fix(KB-043): fix type errors and test failures across all packages

- Add mergeRetries field to updateTask type signature alongside model overrides
- Fix GitManagerModal type imports and error handling
- Update Column, ListView, and GitManagerModal tests for type compatibility
- Fix tsconfig.app.json path references
- Adjust CLI build settings for proper bundling
This commit is contained in:
gsxdsm
2026-03-29 19:36:44 -07:00
parent a5393463d5
commit e06198c6aa
7 changed files with 22 additions and 12 deletions

View File

@@ -25,10 +25,12 @@ function makeTask(id: string): Task {
column: "triage" as ColumnType,
status: undefined as any,
steps: [],
currentStep: 0,
dependencies: [],
description: "",
created: new Date().toISOString(),
updated: new Date().toISOString(),
log: [],
createdAt: new Date().toISOString(),
updatedAt: new Date().toISOString(),
};
}

View File

@@ -37,8 +37,8 @@ import {
const mockAddToast = vi.fn();
const mockTasks: Task[] = [
{ id: "KB-001", description: "Test task 1", column: "in-progress", dependencies: [], worktree: "/worktrees/kb-001" },
{ id: "KB-002", description: "Test task 2", column: "todo", dependencies: [] },
{ id: "KB-001", description: "Test task 1", column: "in-progress", dependencies: [], worktree: "/worktrees/kb-001", steps: [], currentStep: 0, log: [], createdAt: new Date().toISOString(), updatedAt: new Date().toISOString() },
{ id: "KB-002", description: "Test task 2", column: "todo", dependencies: [], steps: [], currentStep: 0, log: [], createdAt: new Date().toISOString(), updatedAt: new Date().toISOString() },
];
describe("GitManagerModal", () => {

View File

@@ -795,15 +795,15 @@ describe("ListView Column Visibility", () => {
// Uncheck all but one
for (let i = 0; i < checkboxes.length - 1; i++) {
if (checkboxes[i].checked) {
if ((checkboxes[i] as HTMLInputElement).checked) {
fireEvent.click(checkboxes[i]);
}
}
// The last checkbox should be disabled (check the disabled property)
const lastCheckbox = checkboxes[checkboxes.length - 1];
if (lastCheckbox.checked) {
expect(lastCheckbox.disabled).toBe(true);
if ((lastCheckbox as HTMLInputElement).checked) {
expect((lastCheckbox as HTMLInputElement).disabled).toBe(true);
}
});