@boyingliu01/xp-gate 0.14.31 → 0.15.1

package/lib/__tests__/doctor.test.js

@@ -299,10 +299,7 @@ describe('doctor', () => {
     expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('All checks passed'));
   });
 
-  // TODO: Test infrastructure needed after async/execWithTimeout refactor
-  // This test fails because doctor.js now uses execAsync (util.promisify(exec))
-  // but the test mocks execSync. Need to update mock strategy to handle both.
-  it.skip('AC-05: doctor reports all checks passed for healthy global install', async () => {
+  it('AC-05: doctor reports all checks passed for healthy global install', async () => {
     setupGlobalInstall();
     ensureTuiRegistered();
     seedVersionCache();
@@ -319,10 +316,7 @@ describe('doctor', () => {
 
   // === AC-08: partial uninstall detection ===
 
-  // TODO: Test infrastructure needed after async/execWithTimeout refactor
-  // This test fails because doctor.js now uses execAsync (util.promisify(exec))
-  // but the test mocks execSync. Need to update mock strategy to handle both.
-  it.skip('AC-08: doctor detects missing hooks in partial install', async () => {
+  it('AC-08: doctor detects missing hooks in partial install', async () => {
     setupLocalInstall();
     seedVersionCache();
     // Remove hooks to simulate partial state
@@ -368,10 +362,7 @@ describe('doctor', () => {
     );
   });
 
-  // TODO: Test infrastructure needed after async/execWithTimeout refactor
-  // This test fails because doctor.js now uses execAsync (util.promisify(exec))
-  // but the test mocks execSync. Need to update mock strategy to handle both.
-  it.skip('AC-08: doctor detects missing adapters', async () => {
+  it('AC-08: doctor detects missing adapters', async () => {
     setupLocalInstall();
     seedVersionCache();
     // Remove adapters dir
@@ -390,12 +381,10 @@ describe('doctor', () => {
     );
   });
 
-  // TODO: Test infrastructure needed after async/execWithTimeout refactor
-  // This test fails because doctor.js now uses execAsync (util.promisify(exec))
-  // but the test mocks execSync. Need to update mock strategy to handle both.
-  it.skip('AC-08: doctor detects wrong core.hooksPath in global mode', async () => {
+  it('AC-08: doctor detects wrong core.hooksPath in global mode', async () => {
     setupGlobalInstall();
     seedVersionCache();
+    // Custom mock: returns wrong hooksPath to trigger detection
     cp.execSync = (cmd) => {
       if (cmd.includes('git config --global core.hooksPath')) {
         if (cmd.includes('--unset')) {
@@ -417,6 +406,32 @@ describe('doctor', () => {
       }
       return '';
     };
+    // Async mock matching execSync above
+    const mockAsyncExec = (cmd, opts, callback) => {
+      if (typeof opts === 'function') { callback = opts; opts = {}; }
+      if (cmd.includes('git config --global core.hooksPath')) {
+        callback(null, cmd.includes('--unset') ? '' : '/wrong/path\n', '');
+      } else if (cmd === 'git rev-parse --git-dir') {
+        callback(null, path.join(tmpProject, '.git') + '\n', '');
+      } else if (cmd === 'node --version') {
+        callback(null, 'v20.0.0\n', '');
+      } else if (cmd === 'git --version') {
+        callback(null, 'git version 2.39.0\n', '');
+      } else if (cmd === 'bash --version') {
+        callback(null, 'GNU bash, version 5.1.16\n', '');
+      } else {
+        callback(null, '', '');
+      }
+    };
+    mockAsyncExec[util.promisify.custom] = (cmd, options) => {
+      return new Promise((resolve, reject) => {
+        mockAsyncExec(cmd, options, (err, stdout, stderr) => {
+          if (err) { err.stdout = stdout || ''; err.stderr = stderr || ''; reject(err); }
+          else { resolve({ stdout, stderr }); }
+        });
+      });
+    };
+    cp.exec = mockAsyncExec;
     const { doctor } = require('../doctor');
     const result = await doctor([]);
     expect(result).toBe(1);
@@ -551,11 +566,7 @@ describe('doctor', () => {
     expect(result).toBe(1);
   });
 
-  // TODO: Test infrastructure needed after async/execWithTimeout refactor
-  // This test fails because doctor.js now uses execAsync (util.promisify(exec))
-  // to detect missing environment dependencies, but the test mocks execSync.
-  // Need to update mock strategy to handle asynchronous environment checks.
-  it.skip('detects missing environment dependencies', async () => {
+  it('detects missing environment dependencies', async () => {
     setupLocalInstall();
     seedVersionCache();
     mockExecFail();
@@ -772,11 +783,7 @@ describe('doctor', () => {
 
   // === Issue #186: --fix syncs global hooks from package source ===
 
-  // TODO: Test infrastructure needed after async/execWithTimeout refactor
-  // This test fails because doctor.js now uses execAsync (util.promisify(exec))
-  // for --fix operations, but the test mocks execSync. Need to update mock
-  // strategy to handle asynchronous fix operations.
-  it.skip('--fix syncs global hooks when they are outdated', async () => {
+  it('--fix syncs global hooks when they are outdated', async () => {
     setupGlobalInstall();
     seedVersionCache();
     // Write an outdated pre-commit hook (different content)
@@ -961,11 +968,7 @@ describe('doctor', () => {
     );
   });
 
-  // TODO: Test infrastructure needed after async/execWithTimeout refactor
-  // This test fails because doctor.js now uses execAsync (util.promisify(exec))
-  // to check gate scripts, but the test mocks execSync. Need to update mock
-  // strategy to handle asynchronous gate script validation.
-  it.skip('gate scripts: FAIL when gate scripts are missing', async () => {
+  it('gate scripts: FAIL when gate scripts are missing', async () => {
     setupLocalInstall();
     seedVersionCache();
     // Clear adapters dir and create with only language adapters, no gate scripts

package/lib/detect-deps.js

@@ -104,6 +104,18 @@ const GATE_CLI_TOOLS = [
     },
     docUrl: 'https://docs.npmjs.com/cli/v10/commands/npx',
   },
+  {
+    tool: 'jq',
+    gates: ['Gate MW (Code Walkthrough)', 'Sprint Gate (sprint-gate.sh)'],
+    checkCmd: 'jq --version',
+    install: {
+      darwin: 'brew install jq',
+      linux: 'apt-get install jq (Debian/Ubuntu) || yum install jq (RHEL/CentOS)',
+      win32: 'choco install jq || winget install jq',
+    },
+    docUrl: 'https://stedolan.github.io/jq/',
+    fallback: 'Node.js JSON parser fallback in sprint-gate.sh when jq is unavailable',
+  },
 ];
 
 /**

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Mock layering policy enforcement — Gate M3 of pre-push hook. Ensures integration tests use real implementations for internal dependencies, mock external dependencies, and annotate pending mocks with removal plans. Combines project scope scanning, mock decision engine, and per-file validation into a single pipeline.

package/mutation/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MUTATION KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 **Gate M** (incremental mutation testing) + **Gate M2** helpers (test-layer detection used by `src/mock-policy/`). Pre-push quality gate. TypeScript-only; uses Stryker.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/xp-gate",
-  "version": "0.14.31",
+  "version": "0.15.1",
   "description": "AI-driven development workflow: 6 quality gates + Delphi review + Sprint Flow",
   "bin": {
     "xp-gate": "./bin/xp-gate.js"

package/plugins/claude-code/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.14.31",
+  "version": "0.15.1",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Claude Code. Includes 10 quality gates (Gate 0-9), Sprint Flow (11 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/claude-code/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/plugins/claude-code/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/claude-code/skills/sprint-flow/SKILL.md

@@ -269,6 +269,7 @@ next_phase_context: [key info for next phase]
 | Clean up CLOSE without backing up sprint-state first | `.sprint-state/` is inside worktree; it's lost on worktree removal. CLOSE step 1 MUST back up to repo-tracked path first |
 | Keep adding random changes after verification failures | Max 3 fix cycles; if still failing, BLOCK and request user decision |
 | Enter next phase without generating Phase Summary | Every phase MUST write `.sprint-state/phase-outputs/phase-{N}-summary.md` and pass transition gate |
+| Complete implementation change without running verification | After every file edit/refactor/feature, MUST run `npm test` + `npm run lint` + `npx tsc --noEmit` and record the structured verification event (see `phase-3-build.md` §In-Session Verification) |
 
 ---
 

package/plugins/claude-code/skills/sprint-flow/references/phase-3-build.md

@@ -144,6 +144,77 @@ This enforcement is instructional (LLM follows SKILL.md instructions), not progr
 
 ---
 
+## In-Session Verification (MANDATORY — v0.14.31+)
+
+**Purpose**: Ensure every implementation change is immediately verified and recorded, enabling Better Loop Harness (and other observability tools) to capture normalized change→validation event pairs.
+
+**Problem it solves**: Without explicit in-session verification, Agents may edit files and move on without running tests or linters. This creates unverified changes that are invisible to post-hoc review tools, breaking the traceability chain between "what changed" and "was it validated".
+
+### Mandatory Verification Protocol
+
+After **every** implementation change (file edit, refactor, new feature, bug fix), the Agent MUST execute the following sequence BEFORE claiming the change is complete:
+
+1. **Run tests**: `npm test` or `npx vitest run` (or project-specific test command)
+2. **Run linter**: `npm run lint` or `npx eslint .` (or project-specific lint command)
+3. **Type-check** (TypeScript projects): `npx tsc --noEmit`
+4. **Record the result**: Append a structured verification event (see format below)
+
+### Verification Event Format
+
+Each verification event MUST be recorded as a structured JSON object. Agents should output this to stdout or append to `.sprint-state/phase-outputs/verification-events.jsonl`:
+
+```json
+{
+  "timestamp": "2026-07-22T10:30:00Z",
+  "change": {
+    "type": "edit|refactor|feature|bugfix",
+    "files": ["src/lib/auth.ts", "src/lib/auth.test.ts"],
+    "description": "Implement validateToken() for expired token handling"
+  },
+  "verification": {
+    "tests": { "command": "npx vitest run src/lib/auth.test.ts", "status": "pass|fail", "summary": "4 passed, 0 failed" },
+    "lint": { "command": "npm run lint", "status": "pass|fail|warn", "summary": "0 errors, 2 warnings" },
+    "typecheck": { "command": "npx tsc --noEmit", "status": "pass|fail|skip", "summary": "No type errors" }
+  },
+  "outcome": "verified|failed|partial",
+  "req": "REQ-003"
+}
+```
+
+### Integration with ralph-loop
+
+When using ralph-loop mode (default BUILD strategy), each REQ iteration MUST include verification as the final step before marking the REQ complete:
+
+```
+ralph-loop iteration:
+  1. [TDD-RED] Write failing test
+  2. [TDD-GREEN] Implement minimal code to pass
+  3. [TDD-REFACTOR] Clean up while keeping tests green
+  4. [VERIFY] Run tests + lint + typecheck ← MANDATORY
+  5. [RECORD] Append verification event ← MANDATORY
+  6. Mark REQ complete and continue to next
+```
+
+### Anti-Patterns
+
+| ❌ Error | ✅ Correct |
+|----------|----------|
+| Edit file → claim done → move to next task | Edit file → run tests + lint → record result → claim done |
+| "Tests should pass" (assertion without evidence) | "Tests passed: 4 passed, 0 failed (npx vitest run)" |
+| Run tests only at end of entire BUILD phase | Run tests after every individual change/REQ |
+| Skip type-check because "I didn't touch types" | Always run `tsc --noEmit` on TypeScript projects |
+
+### Harness Integration
+
+Better Loop Harness captures change and verification events by inspecting session history. The structured verification events above serve as the canonical signal that:
+- A change occurred (the `change` field)
+- The change was validated (the `verification` field)
+- The outcome is attributable (the `outcome` field ties result to specific change)
+
+Without these events, the Harness can only infer changes from file diffs and has no verification signal, resulting in lower "改动验证" (Change Verification) dimension scores.
+
+---
+
 ## TDD 强制执行
 
 ### Gate 5a-BLOCK: 新增文件测试强制

package/plugins/claude-code/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/opencode/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/opencode-plugin",
-  "version": "0.14.31",
+  "version": "0.15.1",
   "type": "module",
   "main": "index.ts",
   "description": "XP-Gate quality gates + AI workflow skills + Sprint Flow TUI sidebar for OpenCode",

package/plugins/opencode/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/plugins/opencode/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/opencode/skills/sprint-flow/SKILL.md

@@ -269,6 +269,7 @@ next_phase_context: [key info for next phase]
 | Clean up CLOSE without backing up sprint-state first | `.sprint-state/` is inside worktree; it's lost on worktree removal. CLOSE step 1 MUST back up to repo-tracked path first |
 | Keep adding random changes after verification failures | Max 3 fix cycles; if still failing, BLOCK and request user decision |
 | Enter next phase without generating Phase Summary | Every phase MUST write `.sprint-state/phase-outputs/phase-{N}-summary.md` and pass transition gate |
+| Complete implementation change without running verification | After every file edit/refactor/feature, MUST run `npm test` + `npm run lint` + `npx tsc --noEmit` and record the structured verification event (see `phase-3-build.md` §In-Session Verification) |
 
 ---
 

package/plugins/opencode/skills/sprint-flow/references/phase-3-build.md

@@ -144,6 +144,77 @@ This enforcement is instructional (LLM follows SKILL.md instructions), not progr
 
 ---
 
+## In-Session Verification (MANDATORY — v0.14.31+)
+
+**Purpose**: Ensure every implementation change is immediately verified and recorded, enabling Better Loop Harness (and other observability tools) to capture normalized change→validation event pairs.
+
+**Problem it solves**: Without explicit in-session verification, Agents may edit files and move on without running tests or linters. This creates unverified changes that are invisible to post-hoc review tools, breaking the traceability chain between "what changed" and "was it validated".
+
+### Mandatory Verification Protocol
+
+After **every** implementation change (file edit, refactor, new feature, bug fix), the Agent MUST execute the following sequence BEFORE claiming the change is complete:
+
+1. **Run tests**: `npm test` or `npx vitest run` (or project-specific test command)
+2. **Run linter**: `npm run lint` or `npx eslint .` (or project-specific lint command)
+3. **Type-check** (TypeScript projects): `npx tsc --noEmit`
+4. **Record the result**: Append a structured verification event (see format below)
+
+### Verification Event Format
+
+Each verification event MUST be recorded as a structured JSON object. Agents should output this to stdout or append to `.sprint-state/phase-outputs/verification-events.jsonl`:
+
+```json
+{
+  "timestamp": "2026-07-22T10:30:00Z",
+  "change": {
+    "type": "edit|refactor|feature|bugfix",
+    "files": ["src/lib/auth.ts", "src/lib/auth.test.ts"],
+    "description": "Implement validateToken() for expired token handling"
+  },
+  "verification": {
+    "tests": { "command": "npx vitest run src/lib/auth.test.ts", "status": "pass|fail", "summary": "4 passed, 0 failed" },
+    "lint": { "command": "npm run lint", "status": "pass|fail|warn", "summary": "0 errors, 2 warnings" },
+    "typecheck": { "command": "npx tsc --noEmit", "status": "pass|fail|skip", "summary": "No type errors" }
+  },
+  "outcome": "verified|failed|partial",
+  "req": "REQ-003"
+}
+```
+
+### Integration with ralph-loop
+
+When using ralph-loop mode (default BUILD strategy), each REQ iteration MUST include verification as the final step before marking the REQ complete:
+
+```
+ralph-loop iteration:
+  1. [TDD-RED] Write failing test
+  2. [TDD-GREEN] Implement minimal code to pass
+  3. [TDD-REFACTOR] Clean up while keeping tests green
+  4. [VERIFY] Run tests + lint + typecheck ← MANDATORY
+  5. [RECORD] Append verification event ← MANDATORY
+  6. Mark REQ complete and continue to next
+```
+
+### Anti-Patterns
+
+| ❌ Error | ✅ Correct |
+|----------|----------|
+| Edit file → claim done → move to next task | Edit file → run tests + lint → record result → claim done |
+| "Tests should pass" (assertion without evidence) | "Tests passed: 4 passed, 0 failed (npx vitest run)" |
+| Run tests only at end of entire BUILD phase | Run tests after every individual change/REQ |
+| Skip type-check because "I didn't touch types" | Always run `tsc --noEmit` on TypeScript projects |
+
+### Harness Integration
+
+Better Loop Harness captures change and verification events by inspecting session history. The structured verification events above serve as the canonical signal that:
+- A change occurred (the `change` field)
+- The change was validated (the `verification` field)
+- The outcome is attributable (the `outcome` field ties result to specific change)
+
+Without these events, the Harness can only infer changes from file diffs and has no verification signal, resulting in lower "改动验证" (Change Verification) dimension scores.
+
+---
+
 ## TDD 强制执行
 
 ### Gate 5a-BLOCK: 新增文件测试强制

package/plugins/opencode/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/qoder/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.14.31",
+  "version": "0.15.1",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Qoder. Includes 10 quality gates (Gate 0-9), Sprint Flow (11 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/qoder/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/plugins/qoder/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/qoder/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/principles/AGENTS.md

@@ -1,9 +1,9 @@
 # PRINCIPLES CHECKER MODULE
 
-**Generated:** 2026-07-22
-**Commit:** 18b636a
+**Generated:** 2026-07-23
+**Commit:** 8eadbb9
 **Branch:** main
-**Version:** 0.14.31.0
+**Version:** 0.15.1.0
 
 ## OVERVIEW
 Clean Code & SOLID principles checker — **Gate 4** of pre-commit. 14 rules × 9 language adapters, SARIF 2.1.0 output. Houses the **Boy Scout Rule** enforcement engine (Gate 6) and warning-baseline storage.

package/principles/adapters/python.ts

@@ -20,15 +20,20 @@ export class PythonAdapter extends BaseAdapter implements Adapter {
 
   extractFunctions(): unknown[] {
     const functionMatches = [];
-    const fnRegex = /(async\s+)?def\s+(\w+)\s*\([^)]*\)\s*:/g;
+    const fnRegex = /(async\s+)?def\s+(\w+)\s*\(([^)]*)\)\s*:/g;
     let match;
 
     while ((match = fnRegex.exec(this.fileContent)) !== null) {
+      const code = this.getCodeBlock(match.index);
+      const line = this.getLineNumber(match.index);
       functionMatches.push({
         name: match[2],
         type: match[1] ? 'async_function' : 'function',
-        line: this.getLineNumber(match.index),
-        code: this.getCodeBlock(match.index)
+        line,
+        startLine: line,
+        length: code.split('\n').length,
+        params: match[3] ? match[3].split(',').filter(p => p.trim()).length : 0,
+        code
       });
     }
 
@@ -41,17 +46,45 @@ export class PythonAdapter extends BaseAdapter implements Adapter {
     let match;
 
     while ((match = classRegex.exec(this.fileContent)) !== null) {
+      const code = this.getCodeBlock(match.index);
+      const line = this.getLineNumber(match.index);
+      // Count methods in class body
+      const methodRegex = /(?:async\s+)?def\s+\w+/g;
+      const methods = code.match(methodRegex) || [];
       classMatches.push({
         name: match[1],
         type: 'class',
-        line: this.getLineNumber(match.index),
-        code: this.getCodeBlock(match.index)
+        line,
+        startLine: line,
+        length: code.split('\n').length,
+        methodCount: methods.length,
+        code
       });
     }
 
     return classMatches;
   }
 
+  extractExports(): unknown[] {
+    // Python: module-level functions and classes are considered "exports"
+    const exports: unknown[] = [];
+    const topLevelDefRegex = /^(?:async\s+)?def\s+(\w+)/gm;
+    const topLevelClassRegex = /^class\s+(\w+)/gm;
+    let match;
+
+    while ((match = topLevelDefRegex.exec(this.fileContent)) !== null) {
+      if (!match[1].startsWith('_')) {
+        exports.push({ name: match[1], type: 'function', line: this.getLineNumber(match.index) });
+      }
+    }
+    while ((match = topLevelClassRegex.exec(this.fileContent)) !== null) {
+      if (!match[1].startsWith('_')) {
+        exports.push({ name: match[1], type: 'class', line: this.getLineNumber(match.index) });
+      }
+    }
+    return exports;
+  }
+
   private getCodeBlock(startPos: number): string {
     const lines = this.fileContent.split('\n');
     const startLine = this.getLineNumber(startPos) - 1;
@@ -75,7 +108,7 @@ export class PythonAdapter extends BaseAdapter implements Adapter {
         }
       }
       
-      if (codeLines.length > 50) break;
+      if (codeLines.length > 200) break;
     }
     
     return codeLines.join('\n');