@exaudeus/workrail 1.14.2 → 1.15.0

package/dist/v2/durable-core/domain/prompt-renderer.js

@@ -114,6 +114,25 @@ function applyPromptBudget(combinedPrompt) {
     const decoder = new TextDecoder('utf-8');
     return decoder.decode(truncatedBytes) + markerText + omissionNote;
 }
+function buildLoopContextBanner(args) {
+    if (args.loopPath.length === 0 || args.isExitStep)
+        return '';
+    const current = args.loopPath[args.loopPath.length - 1];
+    const iterationLabel = `Iteration ${current.iteration + 1}`;
+    return [
+        `---`,
+        `**LOOP: ${current.loopId} | ${iterationLabel}** — This step repeats intentionally; the workflow is not stuck or broken.`,
+        ``,
+        `Choose the instruction that matches your current task:`,
+        `- **Drafting / updating**: incorporate amendments discovered in previous iterations before writing.`,
+        `- **Auditing / reviewing**: look for what previous passes *missed*, not what they already caught.`,
+        `- **Applying changes**: follow prior findings precisely; don't re-debate settled decisions.`,
+        ``,
+        `Prior iteration work is visible in the **Ancestry Recap** section below (if present).`,
+        `---`,
+        ``,
+    ].join('\n');
+}
 function formatOutputContractRequirements(outputContract) {
     const contractRef = outputContract?.contractRef;
     if (!contractRef)
@@ -123,7 +142,8 @@ function formatOutputContractRequirements(outputContract) {
             return [
                 `Artifact contract: ${index_js_2.LOOP_CONTROL_CONTRACT_REF}`,
                 `Provide an artifact with kind: "wr.loop_control"`,
-                `Fields: loopId (lowercase, delimiter-safe), decision ("continue" | "stop")`,
+                `Required field: decision ("continue" | "stop")`,
+                `Optional field: loopId — omit unless targeting a specific named loop`,
             ];
         default:
             return [
@@ -154,14 +174,16 @@ function renderPendingPrompt(args) {
     const basePrompt = step?.prompt ?? `Pending step: ${args.stepId}`;
     const requireConfirmation = Boolean(step?.requireConfirmation);
     const functionReferences = step?.functionReferences ?? [];
+    const outputContract = step && typeof step === 'object' && 'outputContract' in step
+        ? step.outputContract
+        : undefined;
+    const isExitStep = outputContract?.contractRef === index_js_2.LOOP_CONTROL_CONTRACT_REF;
+    const loopBanner = buildLoopContextBanner({ loopPath: args.loopPath, isExitStep });
     const validationCriteria = step?.validationCriteria;
     const requirements = (0, validation_requirements_extractor_js_1.extractValidationRequirements)(validationCriteria);
     const requirementsSection = requirements.length > 0
         ? `\n\n**OUTPUT REQUIREMENTS:**\n${requirements.map(r => `- ${r}`).join('\n')}`
         : '';
-    const outputContract = step && typeof step === 'object' && 'outputContract' in step
-        ? step.outputContract
-        : undefined;
     const contractRequirements = formatOutputContractRequirements(outputContract);
     const contractSection = contractRequirements.length > 0
         ? `\n\n**OUTPUT REQUIREMENTS (System):**\n${contractRequirements.map(r => `- ${r}`).join('\n')}`
@@ -170,8 +192,35 @@ function renderPendingPrompt(args) {
         (step !== null && step !== undefined && 'notesOptional' in step && step.notesOptional === true);
     const notesSection = isNotesOptional
         ? ''
-        : '\n\n**NOTES REQUIRED (System):** You must include `output.notesMarkdown` documenting what you did and why. Omitting notes will block this step — use the `retryAckToken` to fix and retry.';
-    const enhancedPrompt = basePrompt + requirementsSection + contractSection + notesSection;
+        : '\n\n**NOTES REQUIRED (System):** You must include `output.notesMarkdown` when advancing. ' +
+            'These notes are displayed to the user in a markdown viewer and serve as the durable record of your work. Write them for a human reader.\n\n' +
+            'Include:\n' +
+            '- **What you did** and the key decisions or trade-offs you made\n' +
+            '- **What you produced** — files changed, functions added, test results, specific numbers\n' +
+            '- **Anything notable** — risks, open questions, things you deliberately chose NOT to do and why\n\n' +
+            'Formatting: Use markdown headings, bullet lists, `code references`, and **bold** for emphasis. ' +
+            'Be specific — file paths, function names, counts, not vague summaries. ' +
+            '10–30 lines is ideal. Too short is worse than too long.\n\n' +
+            'Scope: THIS step only — WorkRail concatenates notes across steps automatically. Never repeat previous step notes.\n\n' +
+            'Example of BAD notes:\n' +
+            '> Reviewed the code and found some issues. Made improvements to error handling.\n\n' +
+            'Example of GOOD notes:\n' +
+            '> ## Review: Authentication Module\n' +
+            '> **Files examined:** `src/auth/oauth2.ts`, `src/auth/middleware.ts`, `tests/auth.test.ts`\n' +
+            '>\n' +
+            '> ### Key findings\n' +
+            '> - Token refresh logic in `refreshAccessToken()` silently swallows network errors — changed to propagate as `AuthRefreshError`\n' +
+            '> - Added missing `audience` validation in JWT verification (was accepting any audience)\n' +
+            '> - **3 Critical**, 2 Major, 4 Minor findings total\n' +
+            '>\n' +
+            '> ### Decisions\n' +
+            '> - Did NOT flag the deprecated `passport` import — it\'s used only in the legacy path scheduled for removal in Q2\n' +
+            '> - Recommended extracting token storage into a `TokenStore` interface for testability\n' +
+            '>\n' +
+            '> ### Open questions\n' +
+            '> - Should refresh tokens be rotated on every use? Current impl reuses until expiry.\n\n' +
+            'Omitting notes will block this step — use the `retryAckToken` to fix and retry.';
+    const enhancedPrompt = loopBanner + basePrompt + requirementsSection + contractSection + notesSection;
     if (!args.rehydrateOnly) {
         return (0, neverthrow_1.ok)({ stepId: args.stepId, title: baseTitle, prompt: enhancedPrompt, requireConfirmation });
     }

package/dist/v2/durable-core/domain/reason-model.js

@@ -180,7 +180,7 @@ function reasonToBlocker(reason) {
                 code: 'MISSING_REQUIRED_NOTES',
                 pointer: { kind: 'workflow_step', stepId },
                 message: `Step "${stepId}" requires notes documenting your work (output.notesMarkdown).`,
-                suggestedFix: 'Add output.notesMarkdown with meaningful documentation of what you did and why. Use the retryAckToken to retry without rehydrating.',
+                suggestedFix: 'Add output.notesMarkdown with a detailed recap: what you did, key decisions/trade-offs, what you produced (files, paths, numbers), and anything notable. Use markdown formatting. 10–30 lines. Use the retryAckToken to retry without rehydrating.',
             }))
                 .andThen(ensureBlockerTextBudgets);
         case 'required_capability_unknown':

package/dist/v2/durable-core/schemas/artifacts/loop-control.d.ts

@@ -21,7 +21,7 @@ export declare const LoopControlMetadataV1Schema: z.ZodOptional<z.ZodObject<{
 export type LoopControlMetadataV1 = z.infer<typeof LoopControlMetadataV1Schema>;
 export declare const LoopControlArtifactV1Schema: z.ZodObject<{
     kind: z.ZodLiteral<"wr.loop_control">;
-    loopId: z.ZodString;
+    loopId: z.ZodOptional<z.ZodString>;
     decision: z.ZodEnum<["continue", "stop"]>;
     metadata: z.ZodOptional<z.ZodObject<{
         reason: z.ZodOptional<z.ZodString>;
@@ -41,8 +41,8 @@ export declare const LoopControlArtifactV1Schema: z.ZodObject<{
     }>>;
 }, "strict", z.ZodTypeAny, {
     kind: "wr.loop_control";
-    loopId: string;
     decision: "continue" | "stop";
+    loopId?: string | undefined;
     metadata?: {
         reason?: string | undefined;
         issuesFound?: number | undefined;
@@ -51,8 +51,8 @@ export declare const LoopControlArtifactV1Schema: z.ZodObject<{
     } | undefined;
 }, {
     kind: "wr.loop_control";
-    loopId: string;
     decision: "continue" | "stop";
+    loopId?: string | undefined;
     metadata?: {
         reason?: string | undefined;
         issuesFound?: number | undefined;
@@ -63,4 +63,4 @@ export declare const LoopControlArtifactV1Schema: z.ZodObject<{
 export type LoopControlArtifactV1 = z.infer<typeof LoopControlArtifactV1Schema>;
 export declare function isLoopControlArtifact(artifact: unknown): artifact is LoopControlArtifactV1;
 export declare function parseLoopControlArtifact(artifact: unknown): LoopControlArtifactV1 | null;
-export declare function findLoopControlArtifact(artifacts: readonly unknown[], loopId: string): LoopControlArtifactV1 | null;
+export declare function findLoopControlArtifact(artifacts: readonly unknown[], expectedLoopId: string): LoopControlArtifactV1 | null;

package/dist/v2/durable-core/schemas/artifacts/loop-control.js

@@ -21,7 +21,7 @@ exports.LoopControlMetadataV1Schema = zod_1.z
 exports.LoopControlArtifactV1Schema = zod_1.z
     .object({
     kind: zod_1.z.literal('wr.loop_control'),
-    loopId: zod_1.z.string().min(1).max(64).regex(constants_js_1.DELIMITER_SAFE_ID_PATTERN, 'loopId must be delimiter-safe: [a-z0-9_-]+'),
+    loopId: zod_1.z.string().min(1).max(64).regex(constants_js_1.DELIMITER_SAFE_ID_PATTERN, 'loopId must be delimiter-safe: [a-z0-9_-]+').optional(),
     decision: exports.LoopControlDecisionSchema,
     metadata: exports.LoopControlMetadataV1Schema,
 })
@@ -35,13 +35,15 @@ function parseLoopControlArtifact(artifact) {
     const result = exports.LoopControlArtifactV1Schema.safeParse(artifact);
     return result.success ? result.data : null;
 }
-function findLoopControlArtifact(artifacts, loopId) {
+function findLoopControlArtifact(artifacts, expectedLoopId) {
     for (let i = artifacts.length - 1; i >= 0; i--) {
         const artifact = artifacts[i];
         if (!isLoopControlArtifact(artifact))
             continue;
         const parsed = parseLoopControlArtifact(artifact);
-        if (parsed && parsed.loopId === loopId) {
+        if (!parsed)
+            continue;
+        if (parsed.loopId === undefined || parsed.loopId === expectedLoopId) {
             return parsed;
         }
     }

package/dist/v2/usecases/console-routes.js

@@ -29,6 +29,16 @@ function mountConsoleRoutes(app, consoleService) {
             res.status(status).json({ success: false, error: error.message });
         });
     });
+    app.get('/api/v2/sessions/:sessionId/nodes/:nodeId', async (req, res) => {
+        const { sessionId, nodeId } = req.params;
+        const result = await consoleService.getNodeDetail(sessionId, nodeId);
+        result.match((data) => res.json({ success: true, data }), (error) => {
+            const status = error.code === 'NODE_NOT_FOUND' ? 404
+                : error.code === 'SESSION_LOAD_FAILED' ? 404
+                    : 500;
+            res.status(status).json({ success: false, error: error.message });
+        });
+    });
     const consoleDist = resolveConsoleDist();
     if (consoleDist) {
         app.use('/console', express_1.default.static(consoleDist));

package/dist/v2/usecases/console-service.d.ts

@@ -1,22 +1,27 @@
-import type { ResultAsync } from 'neverthrow';
+import { type ResultAsync } from 'neverthrow';
 import type { DirectoryListingPortV2 } from '../ports/directory-listing.port.js';
 import type { DataDirPortV2 } from '../ports/data-dir.port.js';
 import type { SessionEventLogReadonlyStorePortV2 } from '../ports/session-event-log-store.port.js';
-import type { ConsoleSessionListResponse, ConsoleSessionDetail } from './console-types.js';
+import type { SnapshotStorePortV2 } from '../ports/snapshot-store.port.js';
+import type { PinnedWorkflowStorePortV2 } from '../ports/pinned-workflow-store.port.js';
+import type { ConsoleSessionListResponse, ConsoleSessionDetail, ConsoleNodeDetail } from './console-types.js';
 export interface ConsoleServicePorts {
     readonly directoryListing: DirectoryListingPortV2;
     readonly dataDir: DataDirPortV2;
     readonly sessionStore: SessionEventLogReadonlyStorePortV2;
+    readonly snapshotStore: SnapshotStorePortV2;
+    readonly pinnedWorkflowStore: PinnedWorkflowStorePortV2;
 }
 export declare class ConsoleService {
     private readonly ports;
     constructor(ports: ConsoleServicePorts);
     getSessionList(): ResultAsync<ConsoleSessionListResponse, ConsoleServiceError>;
     getSessionDetail(sessionIdStr: string): ResultAsync<ConsoleSessionDetail, ConsoleServiceError>;
+    getNodeDetail(sessionIdStr: string, nodeId: string): ResultAsync<ConsoleNodeDetail, ConsoleServiceError>;
     private collectSessionSummaries;
     private loadSessionSummary;
 }
 export interface ConsoleServiceError {
-    readonly code: 'ENUMERATION_FAILED' | 'SESSION_LOAD_FAILED';
+    readonly code: 'ENUMERATION_FAILED' | 'SESSION_LOAD_FAILED' | 'NODE_NOT_FOUND';
     readonly message: string;
 }