@tuzi-ince/hi-loop 0.3.1 → 0.3.3

package/src/stage-context.js

@@ -0,0 +1,46 @@
+/**
+ * stage 핸들러 실행 맥락 조립 (loop.js 에서 분리, NFR-5).
+ *
+ * loop.js 는 "다음 stage 를 정하는" 책임만 남기고, "무엇을 핸들러에 넘기는가"는 여기 모은다.
+ * `state` 는 getter/setter 라 핸들러가 재대입해도 loop.js 의 `let state` 에 반영된다
+ * (가정 기록은 순수 함수라 새 객체를 만들기 때문에 이 반영이 필요하다).
+ */
+export function buildStageContext({
+  getState,
+  setState,
+  save,
+  goal,
+  cwd,
+  logger,
+  notify,
+  gate,
+  nextAfterReview,
+  nextAfterCommit,
+  statePath,
+  baselineFiles,
+  designReviewEnabled,
+  ports,
+  opts,
+}) {
+  return {
+    get state() {
+      return getState();
+    },
+    set state(v) {
+      setState(v);
+    },
+    goal,
+    cwd,
+    logger,
+    notify,
+    gate,
+    nextAfterReview,
+    nextAfterCommit,
+    statePath,
+    baselineFiles,
+    designReviewEnabled,
+    save,
+    ports,
+    opts,
+  };
+}

package/src/state.js

@@ -86,10 +86,12 @@ export function statePathFor(cwd) {
  * goal 해시를 쓰는 이유: 같은 goal 이면 같은 경로가 나와야 resume 이 이어지고,
  * 다른 goal 끼리는 충돌하지 않는다. 한국어 goal 도 파일명이 깨지지 않는다.
  */
-export function planPathsFor({ cwd, goal }) {
+export function planPathsFor({ cwd, goal, specPath = null }) {
   const slug = createHash('sha1').update(String(goal)).digest('hex').slice(0, 8);
   return {
-    specPath: existsSync(join(cwd, DEFAULT_SPEC_PATH)) ? `docs/spec-${slug}.md` : DEFAULT_SPEC_PATH,
+    // `--spec` 으로 명시 고정하면 goal 해시로 비켜가지 않는다 — 그게 오라클 고정의 핵심이다.
+    // 사람이 관리하는 표준 문서를 여러 goal 이 같은 오라클로 공유해 문서↔소스 drift 를 막는다.
+    specPath: specPath || (existsSync(join(cwd, DEFAULT_SPEC_PATH)) ? `docs/spec-${slug}.md` : DEFAULT_SPEC_PATH),
     testPath: existsSync(join(cwd, DEFAULT_TEST_PATH)) ? `tests/app-${slug}.test.js` : DEFAULT_TEST_PATH,
   };
 }
@@ -107,11 +109,12 @@ export function createState({
   testCommand,
   maxLoops,
   cwd = process.cwd(),
+  specPath: pinnedSpecPath = null,
   now = new Date().toISOString(),
 }) {
   // 경로는 **한 번만** 정하고 상태에 박아둔다. 매번 다시 계산하면 PLAN 이 만든 파일 때문에
-  // 2회차부터 경로가 밀려난다.
-  const { specPath, testPath } = planPathsFor({ cwd, goal });
+  // 2회차부터 경로가 밀려난다. pinnedSpecPath(--spec)가 있으면 그것으로 고정한다.
+  const { specPath, testPath } = planPathsFor({ cwd, goal, specPath: pinnedSpecPath });
   return {
     version: STATE_VERSION,
     goal,
@@ -125,6 +128,10 @@ export function createState({
     sessionSerial: 1,
     specPath,
     testPath,
+    // 사용자가 스펙 오라클을 고정했는가(--spec). PLAN 이 이 표준 문서를 덮어쓰지 않도록 신호한다.
+    specPinned: Boolean(pinnedSpecPath),
+    // FR-21: 문서 정합 게이트를 이미 거쳤는가. 같은 goal 재개 시 다시 묻지 않도록.
+    reconciled: false,
     specSummary: '',
     lastError: '',
     costUsd: 0,