Consolidate scanner/checker: delete internal/ duplicates, use mfer/ package #22

New Issue

clawbot · 2026-02-09T01:05:39+01:00

clawbot commented

2026-02-09 01:05:39 +01:00

Phase 1 item from #10

There are two parallel implementations:

mfer/scanner.go + mfer/checker.go (typed with FileSize, RelFilePath) — superior versions
internal/scanner/ + internal/checker/ (raw int64, string) — older, less type-safe

The mfer/ package versions should be canonical. Delete internal/scanner/ and internal/checker/, then update the CLI code in cmd/ to use the mfer/ package directly.

This reduces code duplication and ensures a single source of truth for scanning and checking logic.

**Phase 1 item from #10** There are two parallel implementations: - `mfer/scanner.go` + `mfer/checker.go` (typed with `FileSize`, `RelFilePath`) — superior versions - `internal/scanner/` + `internal/checker/` (raw `int64`, `string`) — older, less type-safe The `mfer/` package versions should be canonical. Delete `internal/scanner/` and `internal/checker/`, then update the CLI code in `cmd/` to use the `mfer/` package directly. This reduces code duplication and ensures a single source of truth for scanning and checking logic.

clawbot self-assigned this 2026-02-09 01:05:39 +01:00

clawbot referenced this issue from a commit

2026-02-09 01:07:28 +01:00

Remove unused internal/scanner and internal/checker packages

clawbot referenced a pull request that will close this issue

2026-02-09 01:07:35 +01:00

Consolidate scanner/checker — delete internal/scanner/ and internal/checker/ (closes #22) #27

sneak closed this issue

2026-02-09 02:09:01 +01:00

sneak referenced this issue from a commit

2026-02-09 02:09:03 +01:00

Consolidate scanner/checker — delete internal/scanner/ and internal/checker/ (closes #22) (#27)