@capstone-nomos/cli 0.1.1 → 0.1.3

package/dist/bridge/briefing-notes.js

@@ -0,0 +1,28 @@
+import { existsSync, readFileSync, writeFileSync } from 'node:fs';
+import path from 'node:path';
+// 브리핑으로 프롬프트에 넣은 인계 노트의 id. 제출할 때 "이미 받은 노트"로 서버에 함께 보낸다.
+// 서버는 관련 노트를 전부 확인해야 제출을 받는다(NOTES_UNACKNOWLEDGED). 프롬프트로 전달된 노트는 확인한 것이므로
+// 모델이 id를 옮겨 적게 하지 않고 브릿지가 넣는다 — 모델이 직접 넣는 건 반려 뒤 새로 받은 노트뿐이다.
+// Executor가 작업공간에 쓰고(브리핑 직후), MCP 서버의 submit_artifact가 읽는다. 둘은 다른 프로세스다.
+export const BRIEFING_NOTES_FILE = '.nomos-briefing.json';
+export function writeBriefingNotes(workspaceDir, taskId, noteIds) {
+    writeFileSync(path.join(workspaceDir, BRIEFING_NOTES_FILE), `${JSON.stringify({ taskId, noteIds }, null, 2)}\n`);
+}
+// 다른 태스크의 파일이거나 없으면 빈 목록 — 엉뚱한 태스크의 노트를 확인한 것으로 보내지 않는다.
+export function readBriefingNoteIds(workspaceDir, taskId) {
+    const file = path.join(workspaceDir, BRIEFING_NOTES_FILE);
+    if (!existsSync(file))
+        return [];
+    try {
+        const parsed = JSON.parse(readFileSync(file, 'utf-8'));
+        if (parsed.taskId !== taskId || !Array.isArray(parsed.noteIds))
+            return [];
+        return parsed.noteIds.filter((id) => typeof id === 'string');
+    }
+    catch {
+        return [];
+    }
+}
+export function mergeAcknowledged(briefed, fromModel) {
+    return [...new Set([...briefed, ...(fromModel ?? [])])].sort();
+}

package/dist/bridge/claude-args.js

@@ -18,12 +18,32 @@ export function buildClaudeArgs(options) {
     if (options.model)
         args.push('--model', options.model);
     args.push('--mcp-config', options.mcpConfigPath, '--strict-mcp-config');
+    // 파일 편집은 작업 폴더 안에서 자동 승인한다 — settings.json의 deny(.env·contracts 등)는 여전히 이긴다.
+    // 없으면 모델이 계획만 세우고 "쓰기 권한을 승인해 달라"며 커밋 없이 끝난다(실제로 그랬다).
+    args.push('--permission-mode', 'acceptEdits');
     // 헤드리스에는 권한 프롬프트에 답할 사람이 없다. 이게 없으면 모델이 도구를 고른 뒤
     // "승인 대기"에서 멈춘다 — 도구를 못 찾는 것과 증상이 달라 헷갈리기 쉽다.
     // 가변 인자를 마지막에 두어 뒤에 아무것도 붙지 않게 한다.
-    args.push('--allowedTools', allowedToolNames().join(','));
+    args.push('--allowedTools', [...allowedToolNames(), ...allowedBashRules()].join(','));
     return args;
 }
+// 작업에 필요한 셸 명령만 접두사로 허용한다(커밋·시험 실행). **임의 셸을 열지 말 것** — 셸로 파일을 쓰면
+// settings.json의 Edit deny(.env·contracts 등)를 우회한다. 서버의 제출 검증(V3)이 최종 방어선이지만 로컬에서 먼저 막는다.
+export const ALLOWED_BASH_PREFIXES = [
+    'git status',
+    'git diff',
+    'git add',
+    'git commit',
+    'git log',
+    'git rev-parse',
+    'npm test',
+    'npm run test',
+    'npx vitest',
+    'node --test',
+];
+export function allowedBashRules() {
+    return ALLOWED_BASH_PREFIXES.map((prefix) => `Bash(${prefix}:*)`);
+}
 // 우리 MCP 서버 하나만 담은 설정. --mcp-config로 넘긴다.
 //
 // **토큰을 여기 넣지 않는다.** 이 파일은 Claude Code에 넘기는 설정이고, 자격 증명의 정본은

package/dist/bridge/mcp-server.js

@@ -7,6 +7,7 @@ import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js'
 import { z } from 'zod';
 import { readCredentials, updateAccessToken } from './credentials.js';
 import { NomosClient, PolicyStaleLoopError } from './nomos-client.js';
+import { mergeAcknowledged, readBriefingNoteIds } from './briefing-notes.js';
 import { pushTaskBranch } from './push.js';
 // 자격 증명은 ~/.nomos/credentials가 정본이다(0600). 환경변수는 파일이 없을 때만 쓴다
 // (테스트·일회성 실행용). 둘을 섞으면 재발급 결과가 어디에 저장되는지 흐려진다.
@@ -69,13 +70,17 @@ server.registerTool('claim_task', {
 const workspaceDir = process.env.NOMOS_WORKSPACE_DIR ?? process.cwd();
 server.registerTool('submit_artifact', {
     title: 'Submit an artifact',
-    description: 'Submit finished work for a task you have claimed. Commit your work on the task branch first, then provide the commit sha and every path you changed. This tool pushes the task branch before submitting; if the push is refused (wrong branch, commit not on the task branch, remote has diverged) nothing is submitted and you get the reason. The server re-checks the paths against the ownership rules, so listing paths you did not change (or omitting ones you did) will be rejected or will fail verification later.',
+    description: 'Submit finished work for a task you have claimed. Commit your work on the task branch first, then provide the commit sha and every path you changed. This tool pushes the task branch before submitting; if the push is refused (wrong branch, commit not on the task branch, remote has diverged) nothing is submitted and you get the reason. The server re-checks the paths against the ownership rules, so listing paths you did not change (or omitting ones you did) will be rejected or will fail verification later. If handover notes related to your task were published after you started, the submission is rejected with NOTES_UNACKNOWLEDGED and the notes are returned: read them, change and commit your work if they affect it, then submit again with their ids in acknowledged_note_ids. Notes that were already in your instructions are acknowledged automatically.',
     inputSchema: {
         task_id: z.string().describe('The claimed task id'),
         commit_sha: z.string().describe('Commit sha containing the work'),
         changed_paths: z.array(z.string()).describe('Repo-relative paths changed by this commit'),
+        acknowledged_note_ids: z
+            .array(z.string())
+            .optional()
+            .describe('Ids of handover notes you read after a NOTES_UNACKNOWLEDGED rejection'),
     },
-}, async ({ task_id, commit_sha, changed_paths }) => run(async () => {
+}, async ({ task_id, commit_sha, changed_paths, acknowledged_note_ids }) => run(async () => {
     // push할 브랜치와 막을 브랜치는 서버가 정한다. 모델이 넘긴 값으로 고르지 않는다.
     const briefing = (await client.getBriefing(task_id));
     const push = await pushTaskBranch({
@@ -88,7 +93,13 @@ server.registerTool('submit_artifact', {
         process.stderr.write('[nomos-mcp] origin 없음 — push 건너뜀(서버가 mirror 모드로 로컬에서 읽는다)\n');
     }
     // push가 실패하면(PushRefused) 여기까지 오지 않는다 — 제출하지 않고 run()이 모델에게 사유를 돌려준다.
-    const artifact = await client.submitArtifact(task_id, { commitSha: commit_sha, changedPaths: changed_paths });
+    // 브리핑으로 프롬프트에 들어간 노트는 이미 받은 것이다 — Executor가 작업공간에 남긴 id를 함께 보낸다.
+    const acknowledgedNoteIds = mergeAcknowledged(readBriefingNoteIds(workspaceDir, task_id), acknowledged_note_ids);
+    const artifact = await client.submitArtifact(task_id, {
+        commitSha: commit_sha,
+        changedPaths: changed_paths,
+        ...(acknowledgedNoteIds.length === 0 ? {} : { acknowledgedNoteIds }),
+    });
     return { ...artifact, push };
 }, 'submit_artifact'));
 server.registerTool('publish_note', {

package/dist/executor/cleanup.js

@@ -0,0 +1,18 @@
+import { rmSync } from 'node:fs';
+// 뒷정리용 삭제. Windows에서는 자식 프로세스(claude·vitest)가 끝난 직후에도 폴더가 잠깐 잠겨 있어 EBUSY가 난다.
+// Node의 재시도 옵션으로 잠시 기다렸다 다시 지우고, 그래도 못 지우면 경고만 남기고 넘어간다 —
+// finally에서 던지면 이미 받아 둔 결과(PM 초안·검증 보고)를 오류가 덮는다(pm-worker에서 실제로 그랬다).
+export function removeQuietly(dir, warn = defaultWarn) {
+    try {
+        rmSync(dir, { recursive: true, force: true, maxRetries: 5, retryDelay: 200 });
+        return true;
+    }
+    catch (err) {
+        warn(`임시 폴더를 지우지 못했습니다(남겨 둠): ${dir} — ${err.message}`);
+        return false;
+    }
+}
+// stdout은 MCP 서버에서 JSON-RPC 전용이라 경고는 stderr로 쓴다.
+function defaultWarn(line) {
+    process.stderr.write(`[nomos] 경고: ${line}\n`);
+}

package/dist/executor/cli.js

@@ -20,6 +20,7 @@ import { writeFileSync } from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 import { createInterface } from 'node:readline/promises';
+import { writeBriefingNotes } from '../bridge/briefing-notes.js';
 import { buildMcpConfig } from '../bridge/claude-args.js';
 import { credentialsPath, readCredentials, updateAccessToken, writeCredentials, } from '../bridge/credentials.js';
 import { NomosApiError, NomosClient } from '../bridge/nomos-client.js';
@@ -68,6 +69,8 @@ async function handleTask(client, projectId, task) {
         policyHash: briefing.policyHash,
     });
     log(`작업공간 ${workspace.dir} (브랜치 ${workspace.branch})`);
+    // 프롬프트에 넣는 노트 = 제출 때 "받은 노트"로 함께 보낼 노트. MCP 서버(submit_artifact)가 읽는다.
+    writeBriefingNotes(workspace.dir, task.id, (briefing.notes ?? []).map((n) => n.id));
     if (workspace.createdBranch) {
         await client.reportBranch(task.id, workspace.branch);
     }

package/dist/executor/pm-worker.js

@@ -1,8 +1,9 @@
 import { spawn } from 'node:child_process';
-import { mkdtempSync, rmSync, writeFileSync } from 'node:fs';
+import { mkdtempSync, writeFileSync } from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 import { resolveClaudeCommand } from './runner.js';
+import { removeQuietly } from './cleanup.js';
 // 지침·프롬프트는 길어서(이전 초안이 붙는 수정 요청) 명령줄로 넘기지 않는다 — Windows 명령줄은 32k자에서 잘린다.
 // 지침은 파일, 프롬프트는 stdin. 도구는 전부 끄고(--tools ""), 사용자 MCP·세션 기록도 쓰지 않는다.
 export function buildPmClaudeArgs(job, systemPromptFile) {
@@ -80,7 +81,7 @@ export async function runPmJob(job, run = spawnClaude) {
         return toJobResult(out, job.request.model);
     }
     finally {
-        rmSync(dir, { recursive: true, force: true });
+        removeQuietly(dir);
     }
 }
 // 작업 하나를 가져와 처리한다. 처리했으면 true.

package/dist/executor/prompt.js

@@ -12,6 +12,14 @@ export function buildTaskPrompt(briefing, branch) {
     if (spec) {
         sections.push(`# 명세 ${spec.featureKey} — ${spec.title}\n${spec.content}`);
     }
+    // 직전 제출이 반려됐다 — 같은 태스크 브랜치에 이전 작업이 남아 있으니 이어서 고친다.
+    if (briefing.lastRejection) {
+        sections.push([
+            '# 직전 제출이 대표에게 반려됐다',
+            `사유: ${briefing.lastRejection.reason}`,
+            `반려된 커밋: ${briefing.lastRejection.commitSha ?? '(알 수 없음)'} — 이 브랜치에 이전 작업이 남아 있다. 처음부터 다시 만들지 말고 사유에 맞게 고친 뒤 다시 제출한다.`,
+        ].join('\n'));
+    }
     if (briefing.notesBlock) {
         sections.push(briefing.notesBlock);
     }
@@ -22,7 +30,9 @@ export function buildTaskPrompt(briefing, branch) {
         '- 위 목록 밖의 경로는 수정하지 않는다. 비밀 파일(.env, *.pem, *.key, secrets/)은 절대 건드리지 않는다.',
         '- 작업이 끝나면 커밋하고, submit_artifact에 **실제 커밋 sha와 실제로 바꾼 모든 경로**를 넣는다.',
         '  바꾸지 않은 경로를 적거나 sha를 지어내지 않는다 — 서버가 제출 시점에 다시 검증한다.',
-        '- 다음 사람이 알아야 할 결정이나 함정이 있으면 publish_note로 남긴다.',
+        '- 제출이 NOTES_UNACKNOWLEDGED로 반려되면, 작업 중에 새로 올라온 인계 노트가 함께 온다. 읽고, 작업에 영향이 있으면 고쳐서 커밋한 뒤',
+        '  submit_artifact를 다시 부르며 그 노트 id를 acknowledged_note_ids에 넣는다. 위 [인계 노트]의 노트는 따로 넣지 않아도 된다.',
+        '- 다음 사람이 알아야 할 결정이나 함정이 있으면 publish_note로 남긴다. 다른 사람도 따라야 할 결정(계약·형식)은 DECIDED로 남긴다 — 프로젝트 전체에 전달된다.',
         '- 다른 에이전트의 잘못을 노트에 적지 않는다. 그건 raise_dispute의 몫이다.',
     ].join('\n'));
     sections.push(`# 지금 할 일\n태스크 ${task.id}를 claim_task로 받고 위 규칙에 따라 진행하라.`);

package/dist/executor/verify.js

@@ -5,9 +5,10 @@
 // 여기서 판정하는 것은 "명령이 0으로 끝났는가"뿐이다. 출력을 읽고 해석하지 않는다 —
 // 해석이 들어가는 순간 브릿지마다 결과가 달라지고, 그건 결정적 검증이 아니다.
 import { execFile } from 'node:child_process';
-import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import path from 'node:path';
 import { promisify } from 'node:util';
+import { removeQuietly } from './cleanup.js';
 const run = promisify(execFile);
 // 린트와 테스트는 오래 걸릴 수 있지만 무한정은 아니다. 초과하면 SKIPPED로 남긴다 —
 // FAIL로 적으면 코드가 아니라 기계가 느려서 재시도 횟수를 잃는다.
@@ -80,7 +81,7 @@ export async function runSpecTests(workspaceDir, specTests) {
     }
     const dir = path.join(workspaceDir, SPEC_TEST_DIR);
     // 시험지는 작업 결과가 아니다. 커밋에 섞이면 V3에서 신고 누락으로 잡힌다.
-    rmSync(dir, { recursive: true, force: true });
+    removeQuietly(dir);
     mkdirSync(dir, { recursive: true });
     try {
         const files = specTests.map((t) => {
@@ -101,6 +102,6 @@ export async function runSpecTests(workspaceDir, specTests) {
     }
     finally {
         // 남겨두면 다음 실행에서 모델이 시험지를 읽고 거기에 맞춰 고칠 수 있다.
-        rmSync(dir, { recursive: true, force: true });
+        removeQuietly(dir);
     }
 }

package/dist/executor/workspace.js

@@ -25,9 +25,12 @@ function git(repoPath, args) {
     return execFileSync('git', args, { cwd: repoPath, encoding: 'utf-8' }).trim();
 }
 function branchExists(repoPath, branch) {
+    return refExists(repoPath, `refs/heads/${branch}`);
+}
+function refExists(repoPath, ref) {
     try {
         // stdio를 삼킨다 — 없는 브랜치면 git이 stderr에 fatal을 찍어 로그를 더럽힌다.
-        execFileSync('git', ['rev-parse', '--verify', `refs/heads/${branch}`], {
+        execFileSync('git', ['rev-parse', '--verify', '--quiet', ref], {
             cwd: repoPath,
             stdio: 'ignore',
         });
@@ -43,11 +46,17 @@ export function prepareWorkspace(input) {
     const dir = workspaceDir(input.projectId, input.taskId);
     const branch = input.branchName ?? `task/${input.taskId}`;
     const existed = branchExists(input.repoPath, branch);
+    excludeNomosFiles(input.repoPath);
     if (!existsSync(dir)) {
         mkdirSync(path.dirname(dir), { recursive: true });
+        // 재시도(검증 실패·반려)는 같은 태스크 브랜치에서 **이어서** 고친다. 이 노트북에 브랜치가 없어도(다른 노트북에서 했거나 새로 받은 클론)
+        // origin에 있으면 거기서 이어 간다 — 기본 브랜치에서 새로 따면 이전 작업이 안 보이고, 제출 때 push가 갈라진 이력으로 거부된다.
+        const remote = `refs/remotes/origin/${branch}`;
         const args = existed
             ? ['worktree', 'add', dir, branch]
-            : ['worktree', 'add', '-b', branch, dir, input.baseBranch];
+            : refExists(input.repoPath, remote)
+                ? ['worktree', 'add', '-b', branch, dir, `origin/${branch}`]
+                : ['worktree', 'add', '-b', branch, dir, input.baseBranch];
         git(input.repoPath, args);
     }
     const claudeDir = path.join(dir, '.claude');
@@ -57,6 +66,26 @@ export function prepareWorkspace(input) {
     writeFileSync(path.join(claudeDir, '.nomos-policy.json'), `${JSON.stringify({ policyHash: input.policyHash, taskId: input.taskId, generatedAt: new Date().toISOString() }, null, 2)}\n`);
     return { dir, branch, createdBranch: !existed, settingsPath };
 }
+// Executor가 작업공간에 쓰는 파일(MCP 설정·실행 로그·브리핑 노트·정책 표시·settings.json)이 모델의 `git add`에 섞이지 않게 한다.
+// 커밋에 섞이면 신고 경로와 실제 diff가 어긋나 V3가 FAIL을 낸다. info/exclude는 레포 공통이라 모든 worktree에 걸린다.
+export const NOMOS_LOCAL_FILES = ['/.nomos-mcp.json', '/.nomos-run.log', '/.nomos-briefing.json', '/.claude/.nomos-policy.json', '/.claude/settings.json'];
+export function excludeNomosFiles(repoPath) {
+    try {
+        const commonDir = path.resolve(repoPath, git(repoPath, ['rev-parse', '--git-common-dir']));
+        const file = path.join(commonDir, 'info', 'exclude');
+        mkdirSync(path.dirname(file), { recursive: true });
+        const current = existsSync(file) ? readFileSync(file, 'utf-8') : '';
+        const lines = new Set(current.split(/\r?\n/));
+        const missing = NOMOS_LOCAL_FILES.filter((p) => !lines.has(p));
+        if (missing.length === 0)
+            return;
+        const prefix = current.length === 0 || current.endsWith('\n') ? '' : '\n';
+        writeFileSync(file, `${current}${prefix}# NOMOS Executor가 작업공간에 쓰는 파일\n${missing.join('\n')}\n`);
+    }
+    catch {
+        // 못 써도 작업은 진행한다 — 섞이면 V3가 잡는다.
+    }
+}
 // executor clean — worktree를 정리한다. 남겨두는 게 기본이라 정리는 명시적 명령으로만 한다.
 export function cleanWorkspaces() {
     const pruned = [];
@@ -84,7 +113,8 @@ export function cleanWorkspaces() {
     const root = workspacesRoot();
     if (!existsSync(root))
         return { pruned, removed: null };
-    rmSync(root, { recursive: true, force: true });
+    // 사람이 직접 부르는 명령이라 못 지우면 오류를 그대로 보인다(삼키면 지웠다고 거짓 보고한다). 잠금만 재시도로 넘긴다.
+    rmSync(root, { recursive: true, force: true, maxRetries: 5, retryDelay: 200 });
     return { pruned, removed: root };
 }
 export function headSha(dir) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@capstone-nomos/cli",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "NOMOS 에이전트 CLI — 노트북의 Claude Code를 NOMOS 서버에 연결하고 배정된 태스크를 실행한다",
   "type": "module",
   "bin": {