@llm4ts/shell 2.9.1 → 2.9.2

package/flows/epic-stories.js

@@ -116,7 +116,7 @@ const program = Effect.gen(function* () {
             gates,
             // With a roster, the judge and the verifier are leased per story,
             // away from the executor coding it (ADR 0019).
-            judge: (story, diff, seats) => judgeStory(seats.context.roster?.forRole("judge") ?? reasoningMeter.service, story, diff, contextBudget),
+            judge: (story, diff, seats) => judgeStory(seats.context.roster?.forRole("judge") ?? reasoningMeter.service, story, diff, contextBudget, plan),
             verifyBlocked: (story, need, workDir, seats) => verifyBlockedOn(seats.context.roster?.forRole("verifier") ?? reasoningMeter.service, events, files, plan)(story, need, workDir),
             // A roster waits for its own executors (the story's next lease
             // does); without one, poll the single coder's engine.

package/flows/lib/epic-stories.js

@@ -13,7 +13,7 @@ import { Info } from "@llm4ts/flow/FlowEvents";
 import { BlockedVerdict } from "@llm4ts/flow/Stories";
 import { stableHash } from "@llm4ts/flow/Plan";
 import { ReviewIssue } from "@llm4ts/flow/Review";
-import { StoryPlan, pathsNamedIn } from "@llm4ts/flow/StoryPlan";
+import { StoryPlan, dependenciesOf, pathsNamedIn } from "@llm4ts/flow/StoryPlan";
 import { claude, coderIds, pi } from "@llm4ts/runner/Connectors";
 import { FlowAborted, FlowLlmError, ReviewResult, ScriptUsage, coderFor, lintCommand, mergeReviewResults } from "@llm4ts/runner";
 export const epicUsage = [
@@ -380,17 +380,44 @@ const subBar = (scored, story) => ReviewResult.make({
     })),
     summary: `judge:${story.id}`
 });
-/** The story-level judge over the branch diff, bounded by the character budget. */
-export const judgeStory = (reasoning, story, diff, budget) => judge(reasoning, storyDimensions)
-    .evaluate(Sample.make({
-    query: [
+/**
+ * What the story judge reads beside the diff: the story, and what every
+ * story it depends on provides. Without the dependencies the judge cannot
+ * tell the house pattern from a mistake — it once failed a screen for
+ * importing `paymentsDomain` from `payments.fake.ts`, exactly where the
+ * payments contract story declared it.
+ */
+export const storyJudgeQuery = (story, plan) => {
+    const dependencies = plan === undefined
+        ? []
+        : dependenciesOf(plan, story.id).flatMap((id) => {
+            const dependency = plan.story(id);
+            return dependency === undefined
+                ? []
+                : [`- ${dependency.id}: ${dependency.provides.join("; ") || "(nothing declared)"}`];
+        });
+    return [
         `Story: ${story.title}`,
         story.description,
         "",
         `Provides: ${story.provides.join(", ") || "(none)"}`,
         `Owned paths: ${story.owned.join(", ")}`,
-        `Shared read-only: ${story.sharedReadOnly.join(", ") || "(none)"}`
-    ].join("\n"),
+        `Shared read-only: ${story.sharedReadOnly.join(", ") || "(none)"}`,
+        ...(dependencies.length === 0
+            ? []
+            : [
+                "",
+                "Already merged, from the stories this one depends on (their declared interface):",
+                ...dependencies,
+                "Using these exactly as declared (the same module, the same export) is correct and is",
+                "never a house-style or scope problem, even where the module is a fake transport."
+            ])
+    ].join("\n");
+};
+/** The story-level judge over the branch diff, bounded by the character budget. */
+export const judgeStory = (reasoning, story, diff, budget, plan) => judge(reasoning, storyDimensions)
+    .evaluate(Sample.make({
+    query: storyJudgeQuery(story, plan),
     response: cap(diff, budget).text
 }))
     .pipe(Effect.mapError(FlowLlmError.from), Effect.map((scored) => subBar(scored, story)));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llm4ts/shell",
-  "version": "2.9.1",
+  "version": "2.9.2",
   "description": "Interactive shell and CLI for llm4ts: flow discovery, run-a-flow, and view",
   "license": "MIT",
   "type": "module",
@@ -52,9 +52,9 @@
   "dependencies": {
     "@effect/platform-node": "4.0.0-rc.115",
     "@effect/platform-node-shared": "4.0.0-rc.115",
-    "@llm4ts/core": "2.9.1",
-    "@llm4ts/runner": "2.9.1",
-    "@llm4ts/flow": "2.9.1"
+    "@llm4ts/flow": "2.9.2",
+    "@llm4ts/runner": "2.9.2",
+    "@llm4ts/core": "2.9.2"
   },
   "peerDependencies": {
     "effect": "4.0.0-rc.115"