refactor(cli): split jobs command adapters

Author: kushagrchitkar

docs/intentional-architecture-rewrite-2026-06-27/decision-log.md

@@ -69,6 +69,8 @@ Setup provider fix-command execution is explicitly wired at setup command regist
 
 Automation command adapters are split by user-facing verb. `src/edges/cli/commands/automation/install.ts`, `uninstall.ts`, and `status.ts` each own only their command option shape and service request conversion. Automation rendering lives in `src/edges/cli/commands/automation/render.ts`. The deleted `automation.ts` and `automation-render.ts` catchalls should not return.
 
+Jobs command adapters are split by user-facing verb group. `src/edges/cli/commands/jobs/read.ts` owns list/show request shaping, `logs.ts` owns logs/attach request shaping, and `cancel.ts` owns cancel request shaping. Jobs rendering and formatting live in `render.ts` and `format.ts` under the same folder. The deleted `jobs.ts`, `jobs-render.ts`, and `jobs-format.ts` catchalls should not return.
+
 ### Transcript file mechanics are platform, sync eligibility is service
 
 Claude and Codex transcript-store scanning, raw transcript snapshot reads, line counting, and JSONL timestamp extraction belong under `src/platform/transcripts/`. The sync service owns quiet-window eligibility, ledger reconciliation, cursor decisions, and Absorb handoff over typed transcript candidates and snapshots.

docs/intentional-architecture-rewrite-2026-06-27/status.md

@@ -5,7 +5,7 @@ Branch: `codex/intentional-architecture-rewrite`
 
 ## Current State
 
-The branch has more than 280 committed rewrite commits past `dev`. The worklog records 270 production slices so far.
+The branch has more than 280 committed rewrite commits past `dev`. The worklog records 271 production slices so far.
 
 The diff is broad: more than 490 files changed, with tens of thousands of lines reshaped.
 
@@ -127,6 +127,7 @@ This is no longer a small cleanup branch. It is a real ownership rewrite.
 - Split lifecycle CLI operation adapters into owner-named init, Absorb, Garden, and render files under `src/edges/cli/commands/operations/`, deleting the old command catchall and render compatibility paths.
 - Moved setup provider-fix runtime wiring to setup command registration, so setup agent-choice UI consumes an explicit runner instead of importing platform runtime fallback mechanics.
 - Split automation CLI command adapters into install, uninstall, status, and render owner files, deleting the old automation command/render catchalls.
+- Split jobs CLI command adapters into read, logs, cancel, render, and format owner files, deleting the old jobs command/render/format catchalls.
 - Moved the provider-neutral operation spec contract into `src/shared/operation-spec.ts`, so lifecycle services build specs, job stores persist them, and provider adapters execute them without stores or providers importing lifecycle service internals.
 - Moved worker-lock and sync-lock process ownership/liveness facts out of stores; stores now persist lock files over injected owner PID and liveness contracts while CLI/worker edges provide platform process probes.
 - Moved repeated store atomic-write temp-file mechanics into `src/stores/atomic-write.ts`, removing process-PID temp names from job and sync stores.
@@ -145,12 +146,12 @@ This is no longer a small cleanup branch. It is a real ownership rewrite.
 
 ## Latest Checkpoint
 
-The latest slice split automation CLI command adapters by verb. Install, uninstall, status, and automation rendering now live under `src/edges/cli/commands/automation/`; the old `src/edges/cli/commands/automation.ts` and `automation-render.ts` catchalls are gone.
+The latest slice split jobs CLI command adapters by verb. List/show, logs/attach, cancel, jobs rendering, and jobs formatting now live under `src/edges/cli/commands/jobs/`; the old `src/edges/cli/commands/jobs.ts`, `jobs-render.ts`, and `jobs-format.ts` catchalls are gone.
 
 Verification passed:
 
 - `npm run lint`
-- `npx vitest run test/architecture-automation-update-boundaries.test.ts test/architecture-lifecycle-provider-boundaries.test.ts test/automation.test.ts`
+- `npx vitest run test/architecture-jobs-sync-boundaries.test.ts test/jobs-command.test.ts`
 - `npx vitest run test/architecture-*-boundaries.test.ts`
 - `git diff --check`
 - `npm test`
@@ -175,7 +176,7 @@ Previous full-slice verification also passed:
 
 ## Immediate Next Work
 
-Continue top-down subsystem passes before small leak cleanup. The major loose source buckets for jobs, init, config, wiki, viewer read models, worker entrypoints, serve process lifetime, setup/uninstall terminal UI, wiki file mechanics, automation scheduler mechanics, automation scheduler app composition, automation command adapter ownership, setup instruction runtime composition, provider setup-view ownership, setup provider-fix runtime composition, job provider-runner composition, job-worker process spawning, Absorb source resolver composition, Absorb source contract ownership, prompt loader mechanics, update runtime composition, update notifier ownership, setup runtime composition, sync transcript runtime composition, sync-to-job session lookup, CLI app composition, diagnostic fact contracts, provider-neutral agent runtime contracts, lock process-liveness contracts, operation-spec type ownership, operation-output type ownership, path-equality contract ownership, init prompt-context ownership, config command validation ownership, store atomic-write ownership, review command markdown ownership, lifecycle workflow type ownership, lifecycle workflow verb ownership, lifecycle command adapter verb ownership, path construction ownership, shared helper-contract ownership, wiki command target resolution, cross-wiki health coordination, job service view ownership, and lifecycle/job starter result ownership have now been removed or assigned. Remaining candidates include command files that still own workflow decisions, remaining platform modules that read config/store state directly, lifecycle/job boundary duplication that remains after the big moves, and large files whose size may still reflect mixed ownership.
+Continue top-down subsystem passes before small leak cleanup. The major loose source buckets for jobs, init, config, wiki, viewer read models, worker entrypoints, serve process lifetime, setup/uninstall terminal UI, wiki file mechanics, automation scheduler mechanics, automation scheduler app composition, automation command adapter ownership, jobs command adapter ownership, setup instruction runtime composition, provider setup-view ownership, setup provider-fix runtime composition, job provider-runner composition, job-worker process spawning, Absorb source resolver composition, Absorb source contract ownership, prompt loader mechanics, update runtime composition, update notifier ownership, setup runtime composition, sync transcript runtime composition, sync-to-job session lookup, CLI app composition, diagnostic fact contracts, provider-neutral agent runtime contracts, lock process-liveness contracts, operation-spec type ownership, operation-output type ownership, path-equality contract ownership, init prompt-context ownership, config command validation ownership, store atomic-write ownership, review command markdown ownership, lifecycle workflow type ownership, lifecycle workflow verb ownership, lifecycle command adapter verb ownership, path construction ownership, shared helper-contract ownership, wiki command target resolution, cross-wiki health coordination, job service view ownership, and lifecycle/job starter result ownership have now been removed or assigned. Remaining candidates include command files that still own workflow decisions, remaining platform modules that read config/store state directly, lifecycle/job boundary duplication that remains after the big moves, and large files whose size may still reflect mixed ownership.
 
 ## Decision Log
 

docs/intentional-architecture-rewrite-2026-06-27/worklog.md

@@ -1919,3 +1919,12 @@ Two-hundred-seventieth production slice:
 - Added owner-named automation command files for install, uninstall, status, and rendering under `src/edges/cli/commands/automation/`.
 - Updated automation command registration and tests to import the verb-owned adapters directly.
 - Strengthened automation boundary tests so the old automation command and render catchalls cannot return.
+
+Two-hundred-seventy-first production slice:
+
+- Deleted the catchall `src/edges/cli/commands/jobs.ts` command adapter.
+- Deleted the catchall `src/edges/cli/commands/jobs-render.ts` renderer and `jobs-format.ts` formatter.
+- Added owner-named jobs command files for read, logs, cancel, render, and format under `src/edges/cli/commands/jobs/`.
+- Removed the unused non-stream attach compatibility helper while preserving the real streaming attach command path.
+- Updated jobs command registration and tests to import the verb-owned adapters directly.
+- Strengthened jobs/sync boundary tests so the old jobs command, render, and format catchalls cannot return.

src/edges/cli/commands/jobs.ts

@@ -0,0 +1,34 @@
+import { cancelJob } from "../../../../services/jobs/index.js";
+import type { CancelJobRequest } from "../../../../services/jobs/index.js";
+import {
+  renderCancelJobResult,
+  type JobsCommandResult,
+} from "./render.js";
+
+export interface JobCancelCommandOptions {
+  cwd: string;
+  jobId: string;
+  json?: boolean;
+  now?: () => Date;
+  signalProcess: (pid: number, signal: NodeJS.Signals) => void;
+}
+
+export async function runJobsCancel(
+  options: JobCancelCommandOptions,
+): Promise<JobsCommandResult> {
+  return renderCancelJobResult(
+    await cancelJob(toCancelJobRequest(options)),
+    options.json,
+  );
+}
+
+function toCancelJobRequest(
+  options: JobCancelCommandOptions,
+): CancelJobRequest {
+  return {
+    cwd: options.cwd,
+    jobId: options.jobId,
+    now: options.now,
+    signalProcess: options.signalProcess,
+  };
+}

src/edges/cli/commands/jobs/cancel.ts

@@ -1,5 +1,5 @@
-import { formatTextTable } from "./table.js";
-import type { JobServiceView } from "../../../services/jobs/index.js";
+import { formatTextTable } from "../table.js";
+import type { JobServiceView } from "../../../../services/jobs/index.js";
 
 export function formatJobRows(views: JobServiceView[]): string[] {
   return formatTextTable({

src/edges/cli/commands/jobs-format.ts src/edges/cli/commands/jobs/format.tssrc/edges/cli/commands/jobs-format.ts renamed to src/edges/cli/commands/jobs/format.ts

@@ -0,0 +1,63 @@
+import {
+  readJobLog,
+  streamJobLog,
+} from "../../../../services/jobs/index.js";
+import type {
+  JobLogRequest,
+  StreamJobLogRequest,
+} from "../../../../services/jobs/index.js";
+import {
+  renderJobLog,
+  renderStreamJobLogResult,
+  type JobsCommandResult,
+} from "./render.js";
+
+export interface JobLogCommandOptions {
+  cwd: string;
+  jobId: string;
+  json?: boolean;
+}
+
+export interface JobAttachStreamCommandOptions {
+  cwd: string;
+  jobId: string;
+  json?: boolean;
+  now?: () => Date;
+  isPidAlive: (pid: number) => boolean;
+  write: (chunk: string) => void;
+  pollMs?: number;
+}
+
+export async function runJobsLogs(
+  options: JobLogCommandOptions,
+): Promise<JobsCommandResult> {
+  const result = await readJobLog(toJobLogRequest(options));
+  return renderJobLog(result, options.json);
+}
+
+export async function streamJobsAttach(
+  options: JobAttachStreamCommandOptions,
+): Promise<JobsCommandResult> {
+  const result = await streamJobLog(toStreamJobLogRequest(options));
+  return renderStreamJobLogResult(result, options.json, options.write);
+}
+
+function toJobLogRequest(options: JobLogCommandOptions): JobLogRequest {
+  return {
+    cwd: options.cwd,
+    jobId: options.jobId,
+  };
+}
+
+function toStreamJobLogRequest(
+  options: JobAttachStreamCommandOptions,
+): StreamJobLogRequest {
+  return {
+    cwd: options.cwd,
+    jobId: options.jobId,
+    write: options.write,
+    pollMs: options.pollMs,
+    now: options.now,
+    isPidAlive: options.isPidAlive,
+  };
+}

src/edges/cli/commands/jobs/logs.ts

@@ -0,0 +1,57 @@
+import { listJobs, readJob } from "../../../../services/jobs/index.js";
+import type {
+  JobRequest,
+  JobsRequest,
+} from "../../../../services/jobs/index.js";
+import {
+  renderJobsListResult,
+  renderJobsShowResult,
+  type JobsCommandResult,
+} from "./render.js";
+
+export interface JobsListCommandOptions {
+  cwd: string;
+  json?: boolean;
+  now?: () => Date;
+  isPidAlive: (pid: number) => boolean;
+}
+
+export interface JobByIdCommandOptions {
+  cwd: string;
+  jobId: string;
+  json?: boolean;
+  now?: () => Date;
+  isPidAlive: (pid: number) => boolean;
+}
+
+export async function runJobsList(
+  options: JobsListCommandOptions,
+): Promise<JobsCommandResult> {
+  return renderJobsListResult(
+    await listJobs(toJobsRequest(options)),
+    options.json,
+  );
+}
+
+export async function runJobsShow(
+  options: JobByIdCommandOptions,
+): Promise<JobsCommandResult> {
+  return renderJobsShowResult(await readJob(toJobRequest(options)), options.json);
+}
+
+function toJobsRequest(options: JobsListCommandOptions): JobsRequest {
+  return {
+    cwd: options.cwd,
+    now: options.now,
+    isPidAlive: options.isPidAlive,
+  };
+}
+
+function toJobRequest(options: JobByIdCommandOptions): JobRequest {
+  return {
+    cwd: options.cwd,
+    jobId: options.jobId,
+    now: options.now,
+    isPidAlive: options.isPidAlive,
+  };
+}