this-is-enough 0.1.1 → 0.1.3

package/AGENTS.md

@@ -90,15 +90,15 @@ Run once when adopting this workflow in a new repository.
 - `reqs/INTERRUPT.md` must not exist at bootstrap completion
 
 4) First requirement starts only after bootstrap
-- create `REQ-*` through lifecycle section `A) Create New Requirement`
-- confirm via section `B) Confirm Requirement` (`full` or `mini` mode)
+- create `REQ-*` through lifecycle section `A) Create New Requirement` (after first feedback gate)
+- confirm via section `B) Confirm Requirement` (after codebase-informed second feedback gate, then `full` or `mini` mode)
 
 ---
 
 ## Status Model (Requirement-Level)
 
 Allowed statuses:
-- `DRAFT`: requirement exists and is being drafted (lock is active)
+- `DRAFT`: requirement exists and is being drafted through layered feedback/specification (lock is active)
 - `ACTIVE`: confirmed and currently being executed (lock is active)
 - `PAUSED`: intentionally stopped to work on another requirement (lock not held)
 - `BLOCKED`: cannot progress due to external dependency (lock not held)
@@ -149,6 +149,9 @@ Definitions:
   - else announce no active requirement and that new creation is allowed
 
 5) Requirement confirmation rule
+- Before roadmap creation, two feedback gates are mandatory:
+  - Gate 1 (alignment): summarize user request and receive explicit confirmation that understanding is correct
+  - Gate 2 (specification): after codebase exploration and requirement concretization, receive explicit confirmation again
 - Requirement is confirmed by one of:
   - Full mode: `ROADMAP.md` exists.
   - Mini mode: `REQUEST.md` contains a `Mini Roadmap` section and all criteria below are `yes/no`-exactly satisfied:
@@ -205,6 +208,12 @@ Architecture-impacting change criteria (MUST):
 - `reqs/INBOX.md` is for independent/external requests.
 - Do not mix them.
 
+10) Commit cadence rule
+- Use phase cadence as default (not requirement-end only).
+- Minimum: create at least one commit when completing each phase before marking that phase `DONE`.
+- During phase execution, create checkpoint commits at meaningful progress boundaries (e.g., completed ordered tasks or stable validation checkpoints).
+- Before pause/switch/interrupt transitions, make a checkpoint commit for resumability. If impossible, record reason explicitly in `VALIDATION_LOG.md`.
+
 ---
 
 ## Session Start Checklist (MUST)
@@ -266,30 +275,36 @@ Precondition:
 - `reqs/ACTIVE.md` does not exist
 
 Steps:
-1. create `reqs/REQ-XXXX-<slug>/`
-2. create `REQUEST.md`
-3. create `STATUS.md` with status `DRAFT`
-4. create `reqs/ACTIVE.md` pointing to this requirement with status `DRAFT`
-5. if sourced from `reqs/INBOX.md`, update that item status to `promoted`
+1. summarize the incoming requirement in plain language and list assumptions/open points
+2. request user feedback and obtain explicit alignment confirmation (Gate 1)
+3. if alignment is rejected, revise summary and repeat steps 1-2
+4. create `reqs/REQ-XXXX-<slug>/`
+5. create `REQUEST.md` with aligned scope baseline from Gate 1
+6. create `STATUS.md` with status `DRAFT`
+7. create `reqs/ACTIVE.md` pointing to this requirement with status `DRAFT`
+8. if sourced from `reqs/INBOX.md`, update that item status to `promoted`
 
 ### B) Confirm Requirement (`DRAFT` -> `ACTIVE`)
 Trigger:
-- one of:
-  - Full mode: `ROADMAP.md` created in active requirement
-  - Mini mode: `REQUEST.md` includes `Mini Roadmap` and meets all Mini criteria
+- active requirement is in `DRAFT` after Gate 1 alignment
 
 Steps:
-1. choose confirmation mode (`full` or `mini`)
-2. if `full`: create `ROADMAP.md` (phases, goals, acceptance)
-3. if `mini`: add `Mini Roadmap` in `REQUEST.md` (tasks + acceptance + criteria flags)
-4. update `STATUS.md` to `ACTIVE` and record `roadmap_mode: full|mini`
-5. update `reqs/ACTIVE.md` to `ACTIVE`
+1. perform deep codebase exploration for the requirement scope
+2. map impacted components/files, constraints, risks, and validation approach in `REQUEST.md`
+3. present the codebase-informed requirement specification to user and obtain explicit confirmation (Gate 2)
+4. if Gate 2 feedback requests changes, iterate steps 1-3 until confirmed
+5. choose confirmation mode (`full` or `mini`)
+6. if `full`: create `ROADMAP.md` (phases, goals, acceptance)
+7. if `mini`: add `Mini Roadmap` in `REQUEST.md` (tasks + acceptance + criteria flags)
+8. update `STATUS.md` to `ACTIVE` and record `roadmap_mode: full|mini`
+9. update `reqs/ACTIVE.md` to `ACTIVE`
 
 ### C) Execute Phase (loop)
 For each `PHASE-XX`:
 1. write `phases/PHASE-XX/PLAN.md` with ordered tasks, validation checklist, exit criteria
 2. implement according to plan
 3. append evidence to `phases/PHASE-XX/VALIDATION_LOG.md`
+4. create checkpoint commits at meaningful progress boundaries
 
 ### D) Complete Phase (2 required + 2 conditional)
 A phase is `DONE` only when all required checks pass:
@@ -297,15 +312,17 @@ A phase is `DONE` only when all required checks pass:
 2. `STATUS.md` and `reqs/ACTIVE.md` are updated
 3. if architecture-impacting changes occurred, ADR is added/updated in `docs/adr/`
 4. if architecture-impacting changes occurred, `ARCHITECTURE.md` reflects the current design
+5. at least one phase-related commit exists before phase is marked `DONE`
 
 ### E) Pause or Block Current Requirement
 Use when scope split or dependency wait occurs.
 
 Steps:
 1. write checkpoint summary in current `VALIDATION_LOG.md`
-2. update `STATUS.md` to `PAUSED` or `BLOCKED`
-3. fill inactive-state record fields (`changed_at`, `reason`, `resume_condition`, `next_action`)
-4. remove `reqs/ACTIVE.md` to unlock
+2. create a resumable checkpoint commit (or record why not possible in `VALIDATION_LOG.md`)
+3. update `STATUS.md` to `PAUSED` or `BLOCKED`
+4. fill inactive-state record fields (`changed_at`, `reason`, `resume_condition`, `next_action`)
+5. remove `reqs/ACTIVE.md` to unlock
 
 ### F) Switch to Another Requirement
 Precondition:
@@ -346,9 +363,10 @@ Precondition:
 
 Steps:
 1. append a short checkpoint in current phase `VALIDATION_LOG.md`
-2. create `reqs/INTERRUPT.md` with interrupt metadata and narrow scope
-3. execute only interrupt scope and record validation evidence in `reqs/INTERRUPT.md`
-4. if scope exceeds timebox or becomes non-urgent, close interrupt and move work to normal pause/switch + `REQ-*` flow
+2. create a checkpoint commit before leaving current requirement context (or record why not possible in `VALIDATION_LOG.md`)
+3. create `reqs/INTERRUPT.md` with interrupt metadata and narrow scope
+4. execute only interrupt scope and record validation evidence in `reqs/INTERRUPT.md`
+5. if scope exceeds timebox or becomes non-urgent, close interrupt and move work to normal pause/switch + `REQ-*` flow
 
 ### J) Close Urgent Interrupt (Archive + Resume)
 Precondition:
@@ -526,6 +544,8 @@ ADR metadata for selective lookup (required for new ADRs):
 
 ## Prohibitions (MUST NOT)
 - do not create a new requirement while `reqs/ACTIVE.md` exists (except the explicit interrupt exception path)
+- do not create `REQ-*`/`REQUEST.md` before Gate 1 alignment feedback is explicitly confirmed by user
+- do not create `ROADMAP.md` or `REQUEST.md` Mini Roadmap before Gate 2 codebase-informed feedback is explicitly confirmed by user
 - do not treat requirement as confirmed without either:
   - `ROADMAP.md` (full mode), or
   - valid `REQUEST.md` Mini Roadmap that satisfies all Mini criteria (mini mode)
@@ -544,3 +564,5 @@ ADR metadata for selective lookup (required for new ADRs):
 - do not leave `reqs/INTERRUPT.md` after interrupt close; archive it to `reqs/interrupts/`
 - do not duplicate interrupt detail logs in requirement `VALIDATION_LOG.md`; store details only in interrupt record
 - do not switch requirements without writing pause/block checkpoint + inactive-state record
+- do not mark a phase `DONE` without at least one phase-related commit
+- do not pause/switch/open interrupt with uncommitted transition-critical changes unless the reason is explicitly recorded in `VALIDATION_LOG.md`

package/README.md

@@ -55,6 +55,7 @@ node /path/to/this-is-enough/bin/cli.js init --mode existing --cwd /path/to/repo
 - `--dry-run` (변경 예정만 출력)
 - `--force` (기존 파일 덮어쓰기, `AGENTS.md`는 차이가 있으면 `--force` 없이도 백업 후 갱신)
 - `--yes` (호환용, 기본이 이미 non-interactive)
+- `.gitignore`에 `reqs/*` 규칙이 없으면 자동 추가
 
 ### `doctor`
 
@@ -94,6 +95,7 @@ node /path/to/this-is-enough/bin/cli.js upgrade --cwd /path/to/repo
 - 누락된 `docs/adr`, `reqs`, `reqs/interrupts` 생성
 - 누락된 `ARCHITECTURE.md`, `reqs/INBOX.md` 생성
 - `AGENTS.md`가 다르면 백업(`AGENTS.md.bak.<timestamp>`) 후 갱신
+- `.gitignore`에 `reqs/*` 규칙이 없으면 자동 추가
 
 ## Installed Structure
 

package/bin/cli.js

@@ -41,6 +41,7 @@ function resolveProjectPaths(cwd) {
   return {
     agentsPath: path.join(cwd, "AGENTS.md"),
     architecturePath: path.join(cwd, "ARCHITECTURE.md"),
+    gitignorePath: path.join(cwd, ".gitignore"),
     inboxPath: path.join(cwd, "reqs", "INBOX.md"),
     activePath: path.join(cwd, "reqs", "ACTIVE.md"),
     interruptPath: path.join(cwd, "reqs", "INTERRUPT.md"),
@@ -182,6 +183,35 @@ function syncCoreAgents(filePath, content, options, actions) {
   return { status: "replaced", backupPath };
 }
 
+function ensureGitignoreReqsRule(filePath, options, actions) {
+  const rule = "reqs/*";
+  const ruleRegex = /^\s*reqs\/\*(\s+#.*)?\s*$/m;
+
+  if (!fs.existsSync(filePath)) {
+    actions.push(`create ${filePath} with ${rule}`);
+    if (!options.dryRun) {
+      fs.writeFileSync(filePath, `${rule}\n`, "utf8");
+    }
+    return;
+  }
+
+  const current = fs.readFileSync(filePath, "utf8");
+  if (ruleRegex.test(current)) {
+    actions.push(`skip existing ${rule} in ${filePath}`);
+    return;
+  }
+
+  const normalized = current.replace(/\r\n/g, "\n");
+  const nextContent = normalized.endsWith("\n")
+    ? `${normalized}${rule}\n`
+    : `${normalized}\n${rule}\n`;
+
+  actions.push(`append ${rule} to ${filePath}`);
+  if (!options.dryRun) {
+    fs.writeFileSync(filePath, nextContent, "utf8");
+  }
+}
+
 function timestampForBackup() {
   const now = new Date();
   const p = (n) => String(n).padStart(2, "0");
@@ -220,6 +250,7 @@ function initProject(options) {
     ensureDir(dirPath, options, actions);
   }
 
+  ensureGitignoreReqsRule(paths.gitignorePath, options, actions);
   const agentsSyncResult = syncCoreAgents(paths.agentsPath, readTemplateAgents(), options, actions);
   writeFileIfNeeded(paths.architecturePath, ARCHITECTURE_TEMPLATE, options, actions);
   writeFileIfNeeded(paths.inboxPath, INBOX_TEMPLATE, options, actions);
@@ -407,6 +438,7 @@ function upgradeProject(options) {
   ensureDir(paths.adrDir, options, actions);
   ensureDir(paths.reqsDir, options, actions);
   ensureDir(paths.interruptsDir, options, actions);
+  ensureGitignoreReqsRule(paths.gitignorePath, options, actions);
 
   createFileIfMissing(paths.architecturePath, ARCHITECTURE_TEMPLATE, options, actions);
   createFileIfMissing(paths.inboxPath, INBOX_TEMPLATE, options, actions);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "this-is-enough",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Install and bootstrap the ThisIsEnough AGENTS.md workflow.",
   "license": "MIT",
   "bin": {