Skip to content

Fix TS2300 duplicate formatBytes import in doctor.ts (broken build on main) #1249

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
waleedkadous wants to merge 4 commits into from
Closed

Fix TS2300 duplicate formatBytes import in doctor.ts (broken build on main) #1249

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9c343bd
chore(porch): bugfix-1077 init bugfix
waleedkadous Jul 25, 2026
8945f3d
chore(porch): bugfix-1077 fix phase-transition
waleedkadous Jul 25, 2026
fb09840
[Bugfix] Fix TS2300 duplicate formatBytes import in doctor.ts
waleedkadous Jul 25, 2026
be9cb83
[Bugfix] Update builder thread: PR #1249 created
waleedkadous Jul 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions codev/projects/bugfix-1077-consult-pre-flight-auth-state-/status.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
id: bugfix-1077
title: consult-pre-flight-auth-state-
protocol: bugfix
phase: fix
plan_phases: []
current_plan_phase: null
gates:
pr:
status: pending
iteration: 1
build_complete: false
history: []
started_at: '2026-07-25T12:49:13.351Z'
updated_at: '2026-07-25T12:54:44.131Z'
13 changes: 13 additions & 0 deletions codev/state/task-08Gm_thread.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# Builder task-08Gm — thread

## Task
Fix broken build on main: `packages/codev/src/commands/doctor.ts` imported `formatBytes` twice (from `../lib/migration-backup-audit.js` and `../agent-farm/servers/session-log-sweep.js`), causing TS2300 duplicate identifier. Landed via PR #1243.

## Work log
- Confirmed both imports and both call sites: line 111 (session-log summary, uses `measureSessionLogs` data) and line 847 (migration-backup reclaimable bytes).
- Aliased the session-log-sweep import as `formatBytes as formatLogBytes` and updated line 111 to `formatLogBytes(bytes)`. The migration-backup import and its call site (line 847) are unchanged.
- Verifying with `npx tsc --noEmit` in `packages/codev` before creating the PR.

## PR
- PR #1249 created: https://github.com/cluesmith/codev/pull/1249 — verified with `pnpm exec tsc --noEmit` (clean after building codev-core).
- `afx send architect` failed with NOT_FOUND (workspace not active in Tower from this nested worktree); architect notification could not be delivered via afx.
4 changes: 2 additions & 2 deletions packages/codev/src/commands/doctor.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ import { AGENT_FARM_DIR } from '@cluesmith/codev-core/constants';
import {
measureSessionLogs,
resolveLogRetentionDays,
formatBytes,
formatBytes as formatLogBytes,
} from '../agent-farm/servers/session-log-sweep.js';

const __filename = fileURLToPath(import.meta.url);
Expand Down Expand Up @@ -108,7 +108,7 @@ export function checkSessionLogs(
};
}

const summary = `${files} file(s), ${formatBytes(bytes)}`;
const summary = `${files} file(s), ${formatLogBytes(bytes)}`;
if (bytes < SESSION_LOG_WARN_BYTES) {
return { status: 'ok', files, bytes, retentionDays, summary, retentionNote };
}
Expand Down
Loading