@atolis-hq/wake 0.2.91 → 0.2.93

package/dist/src/adapters/http/ui-data.js

@@ -3,7 +3,7 @@ import { join } from 'node:path';
 import { buildResourceUri } from '../../domain/resource-uri.js';
 import { isTerminalStage } from '../../domain/stages.js';
 import { isWorkItemDeleted, isWorkItemFrozen } from '../../domain/work-item-lifecycle.js';
-import { workflowForProjection, workflowNameForProjection } from '../../domain/workflows.js';
+import { universalQueueStage, workflowForProjection, workflowNameForProjection, } from '../../domain/workflows.js';
 import { readFileLockStatus } from '../../lib/lock.js';
 async function readLockInfo(lockFile, now, processInspector) {
     const status = await readFileLockStatus(lockFile, {
@@ -45,7 +45,7 @@ function deriveCondition(item, lastRun, config) {
         return { condition: 'needs-human', reason: `sentinel ${lastRun?.sentinel ?? stage}` };
     }
     const workflow = workflowForProjection(item, config);
-    const hasRoute = config.stages[stage] !== undefined || workflow?.stages[stage] !== undefined;
+    const hasRoute = stage === universalQueueStage ? workflow !== null : workflow?.stages[stage] !== undefined;
     if (!hasRoute) {
         return { condition: 'error', reason: `no route configured for stage "${stage}"` };
     }
@@ -982,30 +982,33 @@ function redact(value, keyHint = '') {
 export async function buildConfigView(input) {
     const ledger = await input.stateStore.readLedger();
     const runnerHealth = ledger?.runners ?? {};
-    const routingTable = Object.entries(input.config.stages).map(([stage, route]) => {
-        const tier = route.tier ?? input.config.defaultTier;
-        const candidates = input.config.tiers[tier] ?? [];
-        const runnerName = route.runner ?? candidates[0];
-        const runner = runnerName !== undefined ? input.config.runners[runnerName] : undefined;
-        // Full fallback order for the tier (#67), each candidate's current pause
-        // state so the UI can show not just who's active but who Wake would fall
-        // sideways to next, and rotate back to once a pause expires.
-        const candidateHealth = candidates.map((name) => {
-            const health = runnerHealth[name];
-            const pausedUntil = health?.pausedUntil;
-            const paused = pausedUntil !== undefined && Date.parse(pausedUntil) > input.now.getTime();
-            return { runnerName: name, paused, pausedUntil };
+    const routingTable = Object.entries(input.config.workflows).flatMap(([workflow, definition]) => {
+        return Object.entries(definition.stages).map(([stage, route]) => {
+            const tier = route.tier ?? input.config.defaultTier;
+            const candidates = input.config.tiers[tier] ?? [];
+            const runnerName = route.runner ?? candidates[0];
+            const runner = runnerName !== undefined ? input.config.runners[runnerName] : undefined;
+            // Full fallback order for the tier (#67), each candidate's current pause
+            // state so the UI can show not just who's active but who Wake would fall
+            // sideways to next, and rotate back to once a pause expires.
+            const candidateHealth = candidates.map((name) => {
+                const health = runnerHealth[name];
+                const pausedUntil = health?.pausedUntil;
+                const paused = pausedUntil !== undefined && Date.parse(pausedUntil) > input.now.getTime();
+                return { runnerName: name, paused, pausedUntil };
+            });
+            return {
+                workflow,
+                stage,
+                action: route.action,
+                tier,
+                runnerName,
+                runnerKind: runner?.kind,
+                model: runner !== undefined && runner.kind !== 'fake' ? runner.model : undefined,
+                timeoutMs: runner !== undefined && runner.kind !== 'fake' ? runner.timeoutMs : undefined,
+                candidates: candidateHealth,
+            };
         });
-        return {
-            stage,
-            action: route.action,
-            tier,
-            runnerName,
-            runnerKind: runner?.kind,
-            model: runner !== undefined && runner.kind !== 'fake' ? runner.model : undefined,
-            timeoutMs: runner !== undefined && runner.kind !== 'fake' ? runner.timeoutMs : undefined,
-            candidates: candidateHealth,
-        };
     });
     return { config: redact(input.config), routingTable };
 }

package/dist/src/core/tick-runner.js

@@ -374,14 +374,12 @@ export function createTickRunner(deps) {
         const targetResourceUri = input.approvalResolution.targetResourceUri;
         const commentId = input.approvalResolution.triggeringCommentId.replace(/[^a-z0-9]+/gi, '-');
         const reviewBody = reviewerMessageFromApprovalComment(input.approvalResolution.triggeringCommentBody ?? '');
-        // Never enumerate specific rejection reasons (self-approval, merge-method
-        // restrictions, branch protection, ...) by string-matching the merge
-        // actor's error — there are too many, and Wake shouldn't need to know
-        // its provider's vocabulary. Any failure here is permanent enough not to
-        // retry blindly forever, so it becomes a policy block with the actor's
-        // own message attached. approve and autoMerge are independent: one
-        // failing doesn't stop the other from being attempted.
-        const blockedReasons = [];
+        // approve and autoMerge are independent: one failing doesn't stop the
+        // other from being attempted. If autoMerge succeeds, a failed approval is
+        // not a policy block on its own; GitHub can reject self-approval while
+        // still accepting auto-merge for repos that do not require review.
+        const approvalBlockedReasons = [];
+        const autoMergeBlockedReasons = [];
         const approvedEventId = `pr-merge-approved-${commentId}`;
         if (input.mergePolicy.approve &&
             (await deps.stateStore.readEventEnvelope(approvedEventId)) === null) {
@@ -408,10 +406,11 @@ export function createTickRunner(deps) {
                 }));
             }
             catch (error) {
-                blockedReasons.push(`Merge policy blocked the approval step: ${describeMergeActorError(error)}`);
+                approvalBlockedReasons.push(`Merge policy blocked the approval step: ${describeMergeActorError(error)}`);
             }
         }
         const autoMergeEventId = `pr-auto-merge-enabled-${commentId}`;
+        let autoMergeSucceeded = false;
         if (input.mergePolicy.autoMerge &&
             (await deps.stateStore.readEventEnvelope(autoMergeEventId)) === null) {
             try {
@@ -435,14 +434,52 @@ export function createTickRunner(deps) {
                         idempotencyKey: `${input.approvalResolution.triggeringCommentId}:pr-auto-merge`,
                     },
                 }));
+                autoMergeSucceeded = true;
             }
             catch (error) {
-                blockedReasons.push(`Merge policy blocked the auto-merge step: ${describeMergeActorError(error)}`);
+                autoMergeBlockedReasons.push(`Merge policy blocked the auto-merge step: ${describeMergeActorError(error)}`);
             }
         }
+        else if (input.mergePolicy.autoMerge) {
+            autoMergeSucceeded = true;
+        }
+        const blockedReasons = autoMergeSucceeded && input.mergePolicy.autoMerge
+            ? autoMergeBlockedReasons
+            : [...approvalBlockedReasons, ...autoMergeBlockedReasons];
         if (blockedReasons.length > 0) {
             return { blocked: true, reason: blockedReasons.join(' ') };
         }
+        // Approval failed but auto-merge covered it (e.g. GitHub rejecting
+        // self-approval on the bot's own PR) - not a policy block, but the
+        // failure should still leave a visible trail instead of vanishing
+        // silently, since it may also be a real permissions/config problem.
+        if (approvalBlockedReasons.length > 0) {
+            const occurredAt = eventStampNow();
+            await deliverOutboundEvent(createEventEnvelope({
+                eventId: `pr-merge-approval-note-${commentId}`,
+                workItemKey: input.projection.workItemKey,
+                streamScope: 'work-item',
+                direction: 'outbound',
+                sourceSystem: 'wake',
+                sourceEventType: PUBLISH_INTENT_REQUESTED_EVENT,
+                sourceRefs: {
+                    repo: input.projection.issue.repo,
+                    issueNumber: input.projection.issue.number,
+                    resourceUri: targetResourceUri,
+                },
+                occurredAt,
+                ingestedAt: occurredAt,
+                trigger: 'context-only',
+                payload: {
+                    kind: 'status-update',
+                    origin: input.projection.origin ?? 'github',
+                    body: `Approval step did not block the merge, but did not succeed on its own: ${approvalBlockedReasons.join(' ')}`,
+                    idempotencyKey: `${commentId}:pr-merge-approval-note`,
+                    deliveryState: 'PENDING',
+                },
+                derivedHints: { stage: input.projection.wake.stage },
+            }));
+        }
         return { blocked: false };
     }
     // The one deterministic approval transition: /approved, wake:auto, and a
@@ -2054,7 +2091,7 @@ export function createTickRunner(deps) {
                             },
                             payload: {
                                 ...publishIntent.payload,
-                                kind: sentinel === 'DONE' ? 'approval-request' : 'status-update',
+                                kind: 'status-update',
                                 body: sentinel === 'DONE'
                                     ? `${parsedRunnerResult.body}\n\n${prReviewApprovalMarker}`
                                     : `${parsedRunnerResult.body}\n\n<!-- wake:pr-review-changes-requested -->`,

package/dist/src/domain/runner-routing.js

@@ -8,11 +8,6 @@ export function maxConfiguredRunnerTimeoutMs(config) {
             }
         }
     }
-    for (const stageRoute of Object.values(config.stages)) {
-        if (stageRoute.runner !== undefined) {
-            activeRunnerNames.add(stageRoute.runner);
-        }
-    }
     for (const commandRoute of Object.values(config.commands)) {
         if (commandRoute.runner !== undefined) {
             activeRunnerNames.add(commandRoute.runner);

package/dist/src/domain/schema.js

@@ -805,10 +805,6 @@ const wakeConfigBaseSchema = z.object({
             tier: 'standard',
         },
     }),
-    stages: z.record(z.string(), stageRouteSchema).default({
-        queue: { action: 'refine', tier: 'light' },
-        implement: { action: 'implement', tier: 'standard' },
-    }),
     ui: z
         .object({
         enabled: z.boolean().default(false),
@@ -925,7 +921,6 @@ export const wakeWorkflowConfigSchema = wakeConfigBaseSchema.pick({
     workflows: true,
     workflowSelectors: true,
     commands: true,
-    stages: true,
 });
 export const wakeConfigSchema = wakeConfigBaseSchema.superRefine((config, ctx) => {
     const promptsRoot = config.paths.promptsRoot ?? defaultPromptsRoot();

package/dist/src/version.js

@@ -124,4 +124,4 @@ export function resolveWakeVersion(options = {}) {
     }
     return '0.1.0-dev';
 }
-export const wakeVersion = "g260ecd9";
+export const wakeVersion = "g5c69d41";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atolis-hq/wake",
-  "version": "0.2.91",
+  "version": "0.2.93",
   "description": "Local autonomous agent control plane for software development",
   "license": "Apache-2.0",
   "repository": {