peaks-loop 4.0.28 → 4.0.30

package/dist/cli/commands/request-commands.js

@@ -1,107 +1,14 @@
 import { InvalidArgumentError } from 'commander';
-import { allowedStatesForRole, createRequestArtifact, listRequestArtifacts, showRequestArtifact, transitionRequestArtifact, PrerequisitesNotSatisfiedError, LintGateError, TypeSanityViolationError, FileSizeViolationError, VALID_REQUEST_TYPES, isRequestType } from '../../services/artifacts/request-artifact-service.js';
+import { createRequestArtifact, listRequestArtifacts, showRequestArtifact, transitionRequestArtifact, PrerequisitesNotSatisfiedError, LintGateError, TypeSanityViolationError, FileSizeViolationError, VALID_REQUEST_TYPES } from '../../services/artifacts/request-artifact-service.js';
+import { applyPerArtifactFormat, inferArtifactName, parseRequestType, parseRole, parseStateForRole, resolveDefaultFormat, VALID_ROLES, } from './request-format-helpers.js';
 import { ConfirmationRequiredError } from '../../services/mode/mode-enforcement.js';
 import { recordBypass, isBypassLimitReached, MAX_BYPASSES_PER_SESSION } from '../../services/mode/bypass-tracker.js';
 import { lintRequestArtifact } from '../../services/artifacts/artifact-lint-service.js';
 import { getRepairCycleStatus } from '../../services/artifacts/repair-cycle-service.js';
 import { fail, ok } from 'peaks-loop-shared/result';
 import { triggerBestPracticeScan } from '../../services/prd/best-practice-auto-trigger.js';
-import { formatMdCompact } from '../../shared/format-md-compact.js';
+import { refreshCodegraphAfterSlice, } from '../../services/codegraph/codegraph-autorefresh.js';
 import { addJsonOption, getErrorMessage, printResult } from '../cli-helpers.js';
-/**
- * Per-artifact default format. Only PRD and tech-doc are user-review
- * surfaces; all other RD/QA/TXT artifacts default to compact. The
- * `--pretty` / `--compact` flags override uniformly. See tech-doc
- * §1.3.
- */
-const DEFAULT_FORMAT_BY_ARTIFACT = {
-    prd: 'pretty',
-    'tech-doc': 'pretty',
-    'code-review': 'compact',
-    'security-review': 'compact',
-    'perf-baseline': 'compact',
-    'bug-analysis': 'compact',
-    'test-cases': 'compact',
-    'test-reports': 'compact',
-    'security-findings': 'compact',
-    'performance-findings': 'compact',
-    handoff: 'compact'
-};
-function resolveDefaultFormat(artifactName) {
-    return DEFAULT_FORMAT_BY_ARTIFACT[artifactName] ?? 'compact';
-}
-function applyPerArtifactFormat(envelope, override) {
-    if (override === null || envelope === null || typeof envelope !== 'object')
-        return envelope;
-    // The service returns `{ id, sessionId, role, body, ... }` for a
-    // single-artifact show. The per-artifact `body` field is the only
-    // thing that changes; we attach a `format` field to surface the
-    // choice to the caller. Slice 023 (R3) AC6 / AC7.
-    const obj = envelope;
-    if (typeof obj.body === 'string') {
-        return {
-            ...obj,
-            body: override === 'compact' ? formatMdCompact(obj.body) : obj.body,
-            format: override
-        };
-    }
-    return envelope;
-}
-/**
- * Map a request-artifact envelope to a `DEFAULT_FORMAT_BY_ARTIFACT` key.
- * The service returns the path of the artifact in `path`; the role +
- * filename stem gives us the artifact name. Falls back to the role
- * itself when no `path` field is present.
- */
-function inferArtifactName(envelope, role) {
-    if (envelope === null || typeof envelope !== 'object')
-        return role;
-    const obj = envelope;
-    const pathField = typeof obj.path === 'string' ? obj.path : '';
-    // Path looks like `.peaks/_runtime/<sid>/<role>/requests/<file>.md`.
-    // The role is the second-to-last directory; the file stem is the
-    // last segment. For RD role, the artifact is one of {code-review,
-    // security-review, perf-baseline, bug-analysis, tech-doc}; the file
-    // stem usually carries the artifact name (e.g. `tech-doc.md`,
-    // `code-review-002.md`). We strip the trailing `-<digits>` suffix
-    // when present.
-    const stem = pathField.split(/[\\/]/).pop()?.replace(/\.md$/, '') ?? '';
-    if (stem.length > 0) {
-        // Try the stem verbatim first.
-        if (DEFAULT_FORMAT_BY_ARTIFACT[stem] !== undefined)
-            return stem;
-        // Strip a trailing -<digits> (e.g. code-review-002 -> code-review).
-        const trimmed = stem.replace(/-\d+$/, '');
-        if (DEFAULT_FORMAT_BY_ARTIFACT[trimmed] !== undefined)
-            return trimmed;
-        // Last-ditch: match by prefix.
-        for (const key of Object.keys(DEFAULT_FORMAT_BY_ARTIFACT)) {
-            if (stem.startsWith(key))
-                return key;
-        }
-    }
-    return role;
-}
-const VALID_ROLES = ['prd', 'ui', 'rd', 'qa', 'sc'];
-function parseRole(value) {
-    if (!VALID_ROLES.includes(value)) {
-        throw new InvalidArgumentError(`must be one of ${VALID_ROLES.join(', ')}`);
-    }
-    return value;
-}
-function parseStateForRole(role, value) {
-    const allowed = allowedStatesForRole(role);
-    if (!allowed.includes(value)) {
-        throw new InvalidArgumentError(`must be one of ${allowed.join(', ')} for role ${role}`);
-    }
-    return value;
-}
-function parseRequestType(value) {
-    if (!isRequestType(value)) {
-        throw new InvalidArgumentError(`must be one of ${VALID_REQUEST_TYPES.join(', ')}`);
-    }
-    return value;
-}
 export function registerRequestCommands(program, io) {
     const request = program.command('request').description('Manage per-request Peaks role artifacts (PRD / UI / RD / QA)');
     addJsonOption(request
@@ -353,6 +260,10 @@ export function registerRequestCommands(program, io) {
             // unchanged — the existing auto-compact orchestrator continues to
             // refuse dispatch at ratio ≥ 0.95.
             let preCompact = null;
+            // 2026-09-03-codegraph-autorefresh: auto codegraph re-index on the
+            // RD → QA slice-complete boundary. Set only for rd:qa-handoff; null
+            // otherwise. Best-effort and fail-silent — never blocks the transition.
+            let codegraphRefresh = null;
             if (role === 'rd' && newState === 'qa-handoff') {
                 try {
                     const { maybePreCompactCheckpoint } = await import('../../services/compact/request-transition-hook.js');
@@ -369,6 +280,13 @@ export function registerRequestCommands(program, io) {
                     // The hook is best-effort; never block the transition.
                     preCompact = null;
                 }
+                try {
+                    codegraphRefresh = await refreshCodegraphAfterSlice(options.project);
+                }
+                catch {
+                    // The refresh is best-effort; never block the transition.
+                    codegraphRefresh = { refreshed: false, reason: 'unavailable', note: 'auto codegraph refresh failed after transition' };
+                }
             }
             // v2.13.2 AC-4 — auto-regen prd/handoff.md on prd:handed-off success.
             // Only fires when the handoff is missing; existing handoffs are not overwritten.
@@ -410,7 +328,7 @@ export function registerRequestCommands(program, io) {
             // transition is not a slice boundary), preCompact is null and
             // we emit a `preCompactCheckpoint: null` field so LLM callers
             // can branch on it without re-deriving zone membership.
-            printResult(io, ok('request.transition', { ...result, preCompactCheckpoint: preCompact?.data ?? null }, preCompact !== null
+            printResult(io, ok('request.transition', { ...result, preCompactCheckpoint: preCompact?.data ?? null, codegraphRefresh }, preCompact !== null
                 ? [
                     `Pre-compact checkpoint written at ratio=${typeof preCompact.data === 'object' && preCompact.data !== null && 'ratio' in preCompact.data
                         ? String(preCompact.data.ratio)

package/dist/cli/commands/request-format-helpers.d.ts

@@ -0,0 +1,19 @@
+/**
+ * Format / parse helpers extracted from `request-commands.ts`
+ * (slice 2026-09-06-split-batch-b) so the command file stays under the
+ * 800-line cap. Behaviour-preserving verbatim move.
+ */
+import { type RequestArtifactRole, type RequestArtifactState, type RequestType } from '../../services/artifacts/request-artifact-service.js';
+export declare function resolveDefaultFormat(artifactName: string): 'compact' | 'pretty';
+export declare function applyPerArtifactFormat(envelope: unknown, override: 'pretty' | 'compact' | null): unknown;
+/**
+ * Map a request-artifact envelope to a `DEFAULT_FORMAT_BY_ARTIFACT` key.
+ * The service returns the path of the artifact in `path`; the role +
+ * filename stem gives us the artifact name. Falls back to the role
+ * itself when no `path` field is present.
+ */
+export declare function inferArtifactName(envelope: unknown, role: string): string;
+export declare const VALID_ROLES: ReadonlyArray<RequestArtifactRole>;
+export declare function parseRole(value: string): RequestArtifactRole;
+export declare function parseStateForRole(role: RequestArtifactRole, value: string): RequestArtifactState;
+export declare function parseRequestType(value: string): RequestType;

package/dist/cli/commands/request-format-helpers.js

@@ -0,0 +1,102 @@
+/**
+ * Format / parse helpers extracted from `request-commands.ts`
+ * (slice 2026-09-06-split-batch-b) so the command file stays under the
+ * 800-line cap. Behaviour-preserving verbatim move.
+ */
+import { InvalidArgumentError } from 'commander';
+import { allowedStatesForRole, isRequestType, VALID_REQUEST_TYPES, } from '../../services/artifacts/request-artifact-service.js';
+import { formatMdCompact } from '../../shared/format-md-compact.js';
+/**
+ * Per-artifact default format. Only PRD and tech-doc are user-review
+ * surfaces; all other RD/QA/TXT artifacts default to compact. The
+ * `--pretty` / `--compact` flags override uniformly. See tech-doc
+ * §1.3.
+ */
+const DEFAULT_FORMAT_BY_ARTIFACT = {
+    prd: 'pretty',
+    'tech-doc': 'pretty',
+    'code-review': 'compact',
+    'security-review': 'compact',
+    'perf-baseline': 'compact',
+    'bug-analysis': 'compact',
+    'test-cases': 'compact',
+    'test-reports': 'compact',
+    'security-findings': 'compact',
+    'performance-findings': 'compact',
+    handoff: 'compact'
+};
+export function resolveDefaultFormat(artifactName) {
+    return DEFAULT_FORMAT_BY_ARTIFACT[artifactName] ?? 'compact';
+}
+export function applyPerArtifactFormat(envelope, override) {
+    if (override === null || envelope === null || typeof envelope !== 'object')
+        return envelope;
+    // The service returns `{ id, sessionId, role, body, ... }` for a
+    // single-artifact show. The per-artifact `body` field is the only
+    // thing that changes; we attach a `format` field to surface the
+    // choice to the caller. Slice 023 (R3) AC6 / AC7.
+    const obj = envelope;
+    if (typeof obj.body === 'string') {
+        return {
+            ...obj,
+            body: override === 'compact' ? formatMdCompact(obj.body) : obj.body,
+            format: override
+        };
+    }
+    return envelope;
+}
+/**
+ * Map a request-artifact envelope to a `DEFAULT_FORMAT_BY_ARTIFACT` key.
+ * The service returns the path of the artifact in `path`; the role +
+ * filename stem gives us the artifact name. Falls back to the role
+ * itself when no `path` field is present.
+ */
+export function inferArtifactName(envelope, role) {
+    if (envelope === null || typeof envelope !== 'object')
+        return role;
+    const obj = envelope;
+    const pathField = typeof obj.path === 'string' ? obj.path : '';
+    // Path looks like `.peaks/_runtime/<sid>/<role>/requests/<file>.md`.
+    // The role is the second-to-last directory; the file stem is the
+    // last segment. For RD role, the artifact is one of {code-review,
+    // security-review, perf-baseline, bug-analysis, tech-doc}; the file
+    // stem usually carries the artifact name (e.g. `tech-doc.md`,
+    // `code-review-002.md`). We strip the trailing `-<digits>` suffix
+    // when present.
+    const stem = pathField.split(/[\\/]/).pop()?.replace(/\.md$/, '') ?? '';
+    if (stem.length > 0) {
+        // Try the stem verbatim first.
+        if (DEFAULT_FORMAT_BY_ARTIFACT[stem] !== undefined)
+            return stem;
+        // Strip a trailing -<digits> (e.g. code-review-002 -> code-review).
+        const trimmed = stem.replace(/-\d+$/, '');
+        if (DEFAULT_FORMAT_BY_ARTIFACT[trimmed] !== undefined)
+            return trimmed;
+        // Last-ditch: match by prefix.
+        for (const key of Object.keys(DEFAULT_FORMAT_BY_ARTIFACT)) {
+            if (stem.startsWith(key))
+                return key;
+        }
+    }
+    return role;
+}
+export const VALID_ROLES = ['prd', 'ui', 'rd', 'qa', 'sc'];
+export function parseRole(value) {
+    if (!VALID_ROLES.includes(value)) {
+        throw new InvalidArgumentError(`must be one of ${VALID_ROLES.join(', ')}`);
+    }
+    return value;
+}
+export function parseStateForRole(role, value) {
+    const allowed = allowedStatesForRole(role);
+    if (!allowed.includes(value)) {
+        throw new InvalidArgumentError(`must be one of ${allowed.join(', ')} for role ${role}`);
+    }
+    return value;
+}
+export function parseRequestType(value) {
+    if (!isRequestType(value)) {
+        throw new InvalidArgumentError(`must be one of ${VALID_REQUEST_TYPES.join(', ')}`);
+    }
+    return value;
+}