@capstone-nomos/cli 0.1.14 → 0.1.15

package/dist/bridge/briefing-notes.js

@@ -3,29 +3,48 @@ import path from 'node:path';
 // 브리핑으로 프롬프트에 넣은 인계 노트의 id. 제출할 때 "이미 받은 노트"로 서버에 함께 보낸다.
 // 서버는 관련 노트를 전부 확인해야 제출을 받는다(NOTES_UNACKNOWLEDGED). 프롬프트로 전달된 노트는 확인한 것이므로
 // 모델이 id를 옮겨 적게 하지 않고 브릿지가 넣는다 — 모델이 직접 넣는 건 반려 뒤 새로 받은 노트뿐이다.
+// 질문으로 정해진 결정(decisions)도 같다 — 브리핑으로 받은 결정 id를 함께 남기고 제출 때 보낸다(DECISIONS_UNACKNOWLEDGED).
 // 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 writeBriefingNotes(workspaceDir, taskId, noteIds, decisionIds = []) {
+    writeFileSync(path.join(workspaceDir, BRIEFING_NOTES_FILE), `${JSON.stringify({ taskId, noteIds, decisionIds }, null, 2)}\n`);
 }
-// 다른 태스크의 파일이거나 없으면 빈 목록 — 엉뚱한 태스크의 노트를 확인한 것으로 보내지 않는다.
-export function readBriefingNoteIds(workspaceDir, taskId) {
+// 다른 태스크의 파일이거나 없으면 빈 목록 — 엉뚱한 태스크의 노트·결정을 확인한 것으로 보내지 않는다.
+function readBriefingIds(workspaceDir, taskId, key) {
     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))
+        const ids = parsed[key];
+        if (parsed.taskId !== taskId || !Array.isArray(ids))
             return [];
-        return parsed.noteIds.filter((id) => typeof id === 'string');
+        return ids.filter((id) => typeof id === 'string');
     }
     catch {
         return [];
     }
 }
+export function readBriefingNoteIds(workspaceDir, taskId) {
+    return readBriefingIds(workspaceDir, taskId, 'noteIds');
+}
+export function readBriefingDecisionIds(workspaceDir, taskId) {
+    return readBriefingIds(workspaceDir, taskId, 'decisionIds');
+}
 export function mergeAcknowledged(briefed, fromModel) {
     return [...new Set([...briefed, ...(fromModel ?? [])])].sort();
 }
+// submit_artifact가 서버에 보내는 본문. 브리핑으로 받은 노트·결정과 모델이 반려 뒤 확인한 것을 합친다.
+// acknowledgedDecisionIds는 비어 있어도 **항상** 보낸다 — 서버는 이 필드가 없으면 옛 CLI(0.1.14 이하)로 보고 결정 확인을 건너뛴다.
+export function buildSubmitBody(input) {
+    const acknowledgedNoteIds = mergeAcknowledged(readBriefingNoteIds(input.workspaceDir, input.taskId), input.acknowledgedNoteIds);
+    return {
+        commitSha: input.commitSha,
+        changedPaths: input.changedPaths,
+        ...(acknowledgedNoteIds.length === 0 ? {} : { acknowledgedNoteIds }),
+        acknowledgedDecisionIds: mergeAcknowledged(readBriefingDecisionIds(input.workspaceDir, input.taskId), input.acknowledgedDecisionIds),
+    };
+}
 // 지금 작업공간의 태스크 id — 권한 도구가 AskUserQuestion을 어느 태스크의 질문으로 올릴지 정한다. 없으면 null.
 export function readBriefingTaskId(workspaceDir) {
     const file = path.join(workspaceDir, BRIEFING_NOTES_FILE);

package/dist/bridge/mcp-server.js

@@ -7,7 +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, readBriefingTaskId } from './briefing-notes.js';
+import { buildSubmitBody, readBriefingTaskId } from './briefing-notes.js';
 import { PERMISSION_TOOL } from './claude-args.js';
 import { DENY_OTHER, denyUnanswered, handleAskUserQuestion, isHandled } from './permission.js';
 import { pushTaskBranch } from './push.js';
@@ -72,7 +72,7 @@ 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 latest commit sha and every path changed on the task branch (including commits from earlier attempts on this branch). 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.',
+    description: 'Submit finished work for a task you have claimed. Commit your work on the task branch first, then provide the latest commit sha and every path changed on the task branch (including commits from earlier attempts on this branch). 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. Likewise, if decisions were made by answering other tasks\' questions after you started, the submission is rejected with DECISIONS_UNACKNOWLEDGED and the decisions are returned (with any new related notes): follow them, change and commit your work if it differs, then submit again with their ids in acknowledged_decision_ids (and the notes\' ids in acknowledged_note_ids). Decisions 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'),
@@ -81,8 +81,12 @@ server.registerTool('submit_artifact', {
             .array(z.string())
             .optional()
             .describe('Ids of handover notes you read after a NOTES_UNACKNOWLEDGED rejection'),
+        acknowledged_decision_ids: z
+            .array(z.string())
+            .optional()
+            .describe('Ids of decisions you read after a DECISIONS_UNACKNOWLEDGED rejection'),
     },
-}, async ({ task_id, commit_sha, changed_paths, acknowledged_note_ids }) => run(async () => {
+}, async ({ task_id, commit_sha, changed_paths, acknowledged_note_ids, acknowledged_decision_ids }) => run(async () => {
     // push할 브랜치와 막을 브랜치는 서버가 정한다. 모델이 넘긴 값으로 고르지 않는다.
     const briefing = (await client.getBriefing(task_id));
     const push = await pushTaskBranch({
@@ -95,13 +99,15 @@ server.registerTool('submit_artifact', {
         process.stderr.write('[nomos-mcp] origin 없음 — push 건너뜀(서버가 mirror 모드로 로컬에서 읽는다)\n');
     }
     // push가 실패하면(PushRefused) 여기까지 오지 않는다 — 제출하지 않고 run()이 모델에게 사유를 돌려준다.
-    // 브리핑으로 프롬프트에 들어간 노트는 이미 받은 것이다 — Executor가 작업공간에 남긴 id를 함께 보낸다.
-    const acknowledgedNoteIds = mergeAcknowledged(readBriefingNoteIds(workspaceDir, task_id), acknowledged_note_ids);
-    const artifact = await client.submitArtifact(task_id, {
+    // 브리핑으로 프롬프트에 들어간 노트·결정은 이미 받은 것이다 — Executor가 작업공간에 남긴 id를 함께 보낸다.
+    const artifact = await client.submitArtifact(task_id, buildSubmitBody({
+        workspaceDir,
+        taskId: task_id,
         commitSha: commit_sha,
         changedPaths: changed_paths,
-        ...(acknowledgedNoteIds.length === 0 ? {} : { acknowledgedNoteIds }),
-    });
+        ...(acknowledged_note_ids === undefined ? {} : { acknowledgedNoteIds: acknowledged_note_ids }),
+        ...(acknowledged_decision_ids === undefined ? {} : { acknowledgedDecisionIds: acknowledged_decision_ids }),
+    }));
     return { ...artifact, push };
 }, 'submit_artifact'));
 server.registerTool('publish_note', {

package/dist/bridge/nomos-client.js

@@ -44,7 +44,9 @@ export class NomosClient {
     async claimTask(taskId) {
         return (await this.request('POST', `/api/tasks/${taskId}/claim`));
     }
-    async submitArtifact(taskId, input) {
+    async submitArtifact(taskId, 
+    // acknowledgedDecisionIds: MCP submit_artifact는 비어 있어도 항상 보낸다(없으면 서버가 옛 CLI로 보고 결정 확인을 건너뛴다).
+    input) {
         return (await this.request('POST', `/api/tasks/${taskId}/artifacts`, input));
     }
     // 브릿지 단계(V2·V4) 보고. 서버가 판정하는 V1A·V1B·V3는 여기로 보낼 수 없다 — 403이 돌아온다.

package/dist/executor/cli.js

@@ -190,8 +190,8 @@ async function handleTask(client, projectId, task) {
     });
     log(`작업공간 ${workspace.dir} (브랜치 ${workspace.branch})`);
     activeWorkspaces.set(task.id, { dir: workspace.dir, repo: briefing.repo.fullName });
-    // 프롬프트에 넣는 노트 = 제출 때 "받은 노트"로 함께 보낼 노트. MCP 서버(submit_artifact)가 읽는다.
-    writeBriefingNotes(workspace.dir, task.id, (briefing.notes ?? []).map((n) => n.id));
+    // 프롬프트에 넣는 노트·결정 = 제출 때 "받은 것"으로 함께 보낼 노트·결정. MCP 서버(submit_artifact)가 읽는다.
+    writeBriefingNotes(workspace.dir, task.id, (briefing.notes ?? []).map((n) => n.id), (briefing.decisions ?? []).map((d) => d.id));
     if (workspace.createdBranch) {
         await client.reportBranch(task.id, workspace.branch);
     }

package/dist/executor/prompt.js

@@ -42,6 +42,16 @@ export function buildTaskPrompt(briefing, branch, environment) {
             '이 브랜치에 이전 작업이 남아 있으면 이어서 진행한다. 같은 질문을 다시 묻지 않는다.',
         ].join('\n'));
     }
+    // 다른 역할·PM이 질문에 답해 확정한 결정 — 이 태스크가 물은 것이 아니어도 프로젝트 전체가 따른다.
+    // 실행 중에는 끼워 넣지 않는다. 작업 중에 새로 정해진 결정은 제출 때 DECISIONS_UNACKNOWLEDGED로 온다.
+    if (briefing.decisions && briefing.decisions.length > 0) {
+        sections.push([
+            '# 질문으로 정해진 결정',
+            '다른 역할·PM이 질문에 답해 확정한 결정이다. 구현이 이 결정과 다르면 결정을 따른다.',
+            '결정 자체가 틀렸거나 바꿔야 한다고 보면 임의로 바꾸지 말고 AskUserQuestion으로 그 역할에 다시 묻는다.',
+            ...briefing.decisions.map((d) => `- [물은 쪽 ${d.askerRole} · 정한 쪽 ${d.targetRole}${d.decidedByName ? ` ${d.decidedByName}` : ''}] Q: ${d.question}\n  A: ${d.answer ?? ''}`),
+        ].join('\n'));
+    }
     // 사람(그 역할 담당·대표)이 룸에서 보낸 메시지 — 이번 실행이 시작되기 전에 온 것만 들어온다(실행 중에는 끼워 넣지 않는다).
     if (briefing.humanMessages && briefing.humanMessages.length > 0) {
         sections.push([
@@ -67,6 +77,7 @@ export function buildTaskPrompt(briefing, branch, environment) {
         '  바꾸지 않은 경로를 적거나 sha를 지어내지 않는다 — 서버가 제출 시점에 다시 검증한다.',
         '- 제출이 NOTES_UNACKNOWLEDGED로 반려되면, 작업 중에 새로 올라온 인계 노트가 함께 온다. 읽고, 작업에 영향이 있으면 고쳐서 커밋한 뒤',
         '  submit_artifact를 다시 부르며 그 노트 id를 acknowledged_note_ids에 넣는다. 위 [인계 노트]의 노트는 따로 넣지 않아도 된다.',
+        '- 제출이 DECISIONS_UNACKNOWLEDGED로 반려되면, 작업 중에 새로 정해진 결정이 함께 온다. 반영하고 그 id를 acknowledged_decision_ids에 넣어 다시 제출하라(함께 온 노트가 있으면 그 id는 acknowledged_note_ids에).',
         '- 다음 사람이 알아야 할 결정이나 함정이 있으면 publish_note로 남긴다. 다른 사람도 따라야 할 결정(계약·형식)은 DECIDED로 남긴다 — 프로젝트 전체에 전달된다.',
         // 실험(E6): 이 줄이 없으면 FE 에이전트가 명세에 없는 BE 계약을 스스로 정해 DECIDED로 발행했다 — 그 노트가 BE 브리핑에 "결정"으로 들어간다.
         `- 단, 다른 역할이 정해야 하는 계약(그 역할이 만드는 API의 응답 형식·상태 코드·필드 이름 등)은 DECIDED로 남기지 않는다 — 네 역할(${task.teamRole ?? '제한 없음'})이 정할 수 있는 것이 아니다.`,

package/package.json

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