Skip to content

Expand CTO operator tools, UI, and tests #88

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

Merged
arul28 merged 7 commits into from
Mar 27, 2026
Merged

Expand CTO operator tools, UI, and tests #88

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
029cd15
Expand CTO operator tools, UI, and tests
arul28 Mar 27, 2026
a1f49a4
Add awaiting_lane_choice & various fixes
arul28 Mar 27, 2026
c229f01
Multiple fixes: sync, queue, utils, PR detail
arul28 Mar 27, 2026
2d4809c
Fix flaky handoff test: use vi.waitFor instead of fixed 25ms delay
arul28 Mar 27, 2026
86d9c58
Enhance terminals, chat, queue, dispatcher
arul28 Mar 27, 2026
614ee23
Fix workflow handoff persistence and tool guards
arul28 Mar 27, 2026
c3b8bf3
Add sync chat controls and tighten workflow guards
arul28 Mar 27, 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
9 changes: 8 additions & 1 deletion .ade/ade.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
version: 1
processes: []
processes:
- id: ad55deza
name: Dev
command:
- npm
- run
- dev
cwd: apps/desktop
stackButtons: []
testSuites: []
laneOverlayPolicies: []
Expand Down
17 changes: 7 additions & 10 deletions .ade/cto/identity.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,6 @@
name: CTO
version: 1
persona: >-
You are the CTO for this project inside ADE.

You are the persistent technical lead who owns architecture, execution
quality, engineering continuity, and team direction.

Use ADE's tools and project context to help the team move forward with clear,
concrete decisions.
version: 3
persona: Persistent project CTO with strategic personality.
personality: strategic
modelPreferences:
provider: claude
Expand All @@ -28,4 +21,8 @@ openclawContextPolicy:
- secret
- token
- system_prompt
updatedAt: 1970-01-01T00:00:00.000Z
onboardingState:
completedSteps:
- identity
completedAt: 2026-03-26T18:45:21.214Z
updatedAt: 2026-03-26T18:45:21.216Z
4 changes: 4 additions & 0 deletions .gitleaks.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,8 @@ paths = [
'''dist/''',
'''release/''',
'''\.git/''',
'''\.test\.ts$''',
'''\.test\.tsx$''',
'''\.spec\.ts$''',
'''\.spec\.tsx$''',
]
10 changes: 5 additions & 5 deletions apps/desktop/src/main/main.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import { app, BrowserWindow, nativeImage, shell } from "electron";
import { execSync } from "node:child_process";
import path from "node:path";
type NodePtyType = typeof import("node-pty");

Check warning on line 4 in apps/desktop/src/main/main.ts

View workflow job for this annotation

GitHub Actions / lint-desktop 

`import()` type annotations are forbidden
import { registerIpc } from "./services/ipc/registerIpc";
import { createFileLogger } from "./services/logging/logger";
import { openKvDb } from "./services/state/kvDb";
Expand Down Expand Up @@ -37,7 +38,7 @@
import { createAdeProjectService } from "./services/projects/adeProjectService";
import { createConfigReloadService } from "./services/projects/configReloadService";
import { IPC } from "../shared/ipc";
import type { ProjectInfo } from "../shared/types";
import type { PortLease, ProjectInfo } from "../shared/types";
import type { AppContext } from "./services/ipc/registerIpc";
import fs from "node:fs";
import net from "node:net";
Expand Down Expand Up @@ -402,7 +403,7 @@
err: toErrorMessage(error)
});
const fallbackHtml = encodeURIComponent(
`<html><body style="margin:0;background:#0f0d14;color:#f8f8f2;font-family:monospace;padding:24px;">` +
`<html><body style="margin:0;background:#0f0d14;color:#f8f8f2;font-family:Geist,-apple-system,BlinkMacSystemFont,sans-serif;padding:24px;">` +
`<h2 style="margin:0 0 12px;">ADE failed to load renderer</h2>` +
`<p style="margin:0 0 8px;">URL: ${rendererUrl.replace(/</g, "&lt;")}</p>` +
`<p style="margin:0;">Error: ${toErrorMessage(error).replace(/</g, "&lt;")}</p>` +
Expand Down Expand Up @@ -498,7 +499,7 @@
const loadPty = () => {
// node-pty is a native dependency; keep the require inside the main process runtime.
// eslint-disable-next-line @typescript-eslint/no-var-requires
return require("node-pty") as typeof import("node-pty");
return require("node-pty") as NodePtyType;
};

const normalizeProjectRoot = (projectRoot: string) => path.resolve(projectRoot);
Expand Down Expand Up @@ -671,7 +672,7 @@
logger,
broadcastEvent: (ev) => emitProjectEvent(projectRoot, IPC.lanesPortEvent, ev),
persistLeases: (leases) => db.setJson("port_leases", leases),
loadLeases: () => db.getJson<import("../shared/types").PortLease[]>("port_leases") ?? [],
loadLeases: () => db.getJson<PortLease[]>("port_leases") ?? [],
});
portAllocationService.restore();

Expand Down Expand Up @@ -1240,7 +1241,6 @@
transcriptsDir: adePaths.transcriptsDir,
projectId,
memoryService,
memoryFilesService,
fileService,
workerAgentService,
workerHeartbeatService,
Expand Down
Loading
Loading