@rigour-labs/cli 2.7.0 → 2.8.0

package/dist/commands/constants.d.ts

@@ -1,3 +1,4 @@
 export declare const CODE_QUALITY_RULES = "\n# Code Quality Standards\n\n## PRODUCTION-GRADE CODE ONLY\n- No debug logging in production code\n- No shortcuts or \"temporary\" fixes\n- No over-engineering - simplest solution that works\n- Follow existing code patterns and conventions\n- Handle edge cases properly\n- No TODO/FIXME comments in final code\n\n## MODULAR CODE STRUCTURE\n- Write SMALL, focused functions (< 50 lines ideally)\n- One function = one job, clearly named\n- New features go in SEPARATE FILES, not flooding existing ones\n- Keep files under 500 lines - split if growing larger\n- Extract reusable logic into utility modules\n- Avoid \"god files\" that do everything\n- When adding to existing code, check if a new module is more appropriate\n\n## Technical Standards\n\n### DRY Principle\n- Extract repeated logic into utilities\n- Single Responsibility: One function, one job\n- Defensive coding: Validate inputs at boundaries\n- Lazy initialization for external dependencies (secrets, connections)\n- Graceful degradation over hard failures\n\n### File Organization\n```\n# Good: Separate concerns into focused files\ngovernor/\n  main.py              # Entry point only\n  drift_detector.py    # Drift detection logic\n  lip_sync_analyzer.py # SyncNet integration\n  audio_analyzer.py    # Audio analysis\n\n# Bad: One massive file with everything\ngovernor/\n  main.py (2000+ lines with all logic mixed)\n```\n\n### API Design\n- Consistent error responses\n- Proper HTTP status codes\n- Authentication at the edge\n- Rate limiting on public endpoints\n\n## PRODUCTION-READY SELF-REVIEW (THE GATEKEEPER)\n\nBefore asking for \"approval,\" internally verify:\n\n- **Zero-Dependency Check**: Does this fix rely on a local environment variable not yet in `talentlyt-kv`?\n- **Side-Effect Audit**: Could this change trigger a 502 Bad Gateway at the `/auth/callback` or `/api/agent` endpoints?\n- **Biometric Integrity**: If touching the `Governor`, have I verified that the `similarity_score` logic remains deterministic?\n- **Cost Impact**: Does this change increase egress costs (e.g., unnecessary cross-region logging)?\n- **Error Handling**: Does the UI have a graceful fallback if the backend service is slow?\n";
 export declare const DEBUGGING_RULES = "\n# Investigation & Debugging Protocol\n\n## INVESTIGATION PROTOCOL\n\nWhen debugging:\n1. Check DEPLOYED environment (Azure, prod), not localhost unless explicitly asked\n2. Trace the actual request flow end-to-end\n3. Collect evidence at each step\n4. Present findings before proposing fixes\n\n## GAP ANALYSIS\n\nWhen debugging or proposing changes:\n\n1. **Trace the actual request flow** end-to-end:\n   - Client \u2192 Cloudflare \u2192 Vercel/Container App \u2192 DB\n\n2. **Identify Hidden Gaps** - Explicitly check if the change affects:\n   - **Cross-Region Handshakes**: Will this increase latency for users in Pakistan/India?\n   - **Forensic Continuity**: Does this change how Maya captures gaze or audio data?\n   - **Auth Persistence**: Will this interfere with WorkOS session tokens or M2M keys?\n\n3. **Evidence-First**: Collect logs from `talentlyt-dashboard` before proposing a fix.\n\n## Request Flow Tracing\n\n```\nClient Browser\n    \u2193\nCloudflare (CDN/WAF)\n    \u2193\nAzure Container Apps\n    \u251C\u2500\u2500 talentlyt-dashboard (Next.js)\n    \u2514\u2500\u2500 talentlyt-agent (Python/LiveKit)\n    \u2193\nPostgreSQL Database\n    \u2193\nAzure Blob Storage (recordings, evidence)\n```\n\n## Evidence Collection\n\nBefore proposing any fix:\n1. Get the actual error from logs (not assumed)\n2. Identify the exact file and line number\n3. Trace the data flow that led to the error\n4. Verify the fix doesn't break other paths\n";
 export declare const COLLABORATION_RULES = "\n# Role & Collaboration\n\nYou are a Senior Staff Engineer working alongside a Principal Engineer (the user). \nYou do NOT work autonomously - you work collaboratively with approval at each step.\n\n## 1. NO ASSUMPTIONS\n- Never assume root cause without evidence from logs/code\n- Never assume a fix works without verification\n- Always trace the ACTUAL flow, not the expected flow\n- When debugging, read the DEPLOYED code, not local code\n\n## 2. APPROVAL REQUIRED\nBefore making ANY code change, you MUST:\n1. Show the evidence (logs, code trace) proving the issue\n2. Explain the root cause with proof\n3. Propose the fix with rationale\n4. Wait for explicit approval: \"approved\", \"go ahead\", \"do it\"\n\nException: Only proceed without approval if user explicitly says \"just do it\" or \"fix it\"\n\n## 3. NEVER LOSE TRACK\n- Maintain TODO list for multi-step tasks\n- Complete current task before starting new ones\n- If interrupted, summarize current state before switching\n- Reference previous findings, don't repeat investigations\n\n## Communication\n\n### When Reporting Issues\n```\n**Evidence:** [actual log/error message]\n**Location:** [file:line or endpoint]\n**Root Cause:** [proven, not assumed]\n**Privacy Impact:** [Does this affect biometric/PII data?]\n**Fix:** [proposed solution]\n```\n\n### When Asking for Approval\n```\nI found: [evidence]\nRoot cause: [explanation]\nProposed fix: [code change summary]\n\nApprove to proceed?\n```\n\n### When Stuck\n- Say \"I need more information\" not guess\n- Ask specific questions\n- Propose diagnostic steps\n\n## \u26A0\uFE0F RIGOUR.YML IS SACRED\n\nThe `rigour.yml` file represents **team-agreed quality standards**. You are FORBIDDEN from:\n\n1. \u274C Increasing thresholds to pass checks (e.g., raising `complexity: 100`)\n2. \u274C Adding patterns to `ignore:` to skip problematic files\n3. \u274C Removing items from `required_files`\n4. \u274C Disabling gates to avoid failures\n\n**Your job is to FIX THE CODE to meet the standards, NOT weaken the standards to pass the check.**\n\nIf thresholds genuinely need adjustment, escalate to the team lead with justification.\n\n## Forbidden Actions\n\n1. \u274C Making code changes without showing evidence first\n2. \u274C Testing on localhost when asked to check production\n3. \u274C Adding debug logs as a \"fix\"\n4. \u274C Multiple deployment attempts hoping it works\n5. \u274C Over-engineering simple solutions\n6. \u274C Assuming secrets/env vars are available at init time\n7. \u274C Ignoring user corrections\n8. \u274C Losing context between messages\n9. \u274C Modifying rigour.yml to pass quality checks\n";
+export declare const AGNOSTIC_AI_INSTRUCTIONS = "\n# \uD83E\uDD16 CRITICAL INSTRUCTION FOR AI\n\nWhen asked to implement a feature or fix an issue:\n\n1. **FIRST**: Read ALL relevant rules files (e.g., `.cursor/rules/*.mdc`, `.clinerules`, `docs/AGENT_INSTRUCTIONS.md`).\n2. **NEVER** claim \"done\" until you can demonstrate the feature WORKS or the fix is verified.\n3. **ALWAYS** run `npx @rigour-labs/cli check` before claiming completion.\n4. **CONNECTivity**: \n   - If you create backend code, you MUST wire it to the frontend.\n   - If you create a service, you MUST connect it to the UI.\n\n## \u2705 VERIFICATION PROOF REQUIRED\n\nBefore saying \"done\", show ONE of:\n- Screenshot/output proving it works.\n- The exact commands the user can run to verify.\n- The full flow from initiation (e.g., button click) to result.\n\n\"Code exists\" \u2260 \"Feature works\"\n";

package/dist/commands/constants.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.COLLABORATION_RULES = exports.DEBUGGING_RULES = exports.CODE_QUALITY_RULES = void 0;
+exports.AGNOSTIC_AI_INSTRUCTIONS = exports.COLLABORATION_RULES = exports.DEBUGGING_RULES = exports.CODE_QUALITY_RULES = void 0;
 exports.CODE_QUALITY_RULES = `
 # Code Quality Standards
 
@@ -186,3 +186,24 @@ If thresholds genuinely need adjustment, escalate to the team lead with justific
 8. ❌ Losing context between messages
 9. ❌ Modifying rigour.yml to pass quality checks
 `;
+exports.AGNOSTIC_AI_INSTRUCTIONS = `
+# 🤖 CRITICAL INSTRUCTION FOR AI
+
+When asked to implement a feature or fix an issue:
+
+1. **FIRST**: Read ALL relevant rules files (e.g., \`.cursor/rules/*.mdc\`, \`.clinerules\`, \`docs/AGENT_INSTRUCTIONS.md\`).
+2. **NEVER** claim "done" until you can demonstrate the feature WORKS or the fix is verified.
+3. **ALWAYS** run \`npx @rigour-labs/cli check\` before claiming completion.
+4. **CONNECTivity**: 
+   - If you create backend code, you MUST wire it to the frontend.
+   - If you create a service, you MUST connect it to the UI.
+
+## ✅ VERIFICATION PROOF REQUIRED
+
+Before saying "done", show ONE of:
+- Screenshot/output proving it works.
+- The exact commands the user can run to verify.
+- The full flow from initiation (e.g., button click) to result.
+
+"Code exists" ≠ "Feature works"
+`;

package/dist/commands/init.d.ts

@@ -1,7 +1,7 @@
 export interface InitOptions {
     preset?: string;
     paradigm?: string;
-    ide?: 'cursor' | 'vscode' | 'all';
+    ide?: 'cursor' | 'vscode' | 'cline' | 'all';
     dryRun?: boolean;
     explain?: boolean;
 }

package/dist/commands/init.js

@@ -11,6 +11,10 @@ const yaml_1 = __importDefault(require("yaml"));
 const core_1 = require("@rigour-labs/core");
 const constants_js_1 = require("./constants.js");
 function detectIDE(cwd) {
+    // Check for Cline-specific markers
+    if (fs_extra_1.default.existsSync(path_1.default.join(cwd, '.clinerules'))) {
+        return 'cline';
+    }
     // Check for Cursor-specific markers
     if (fs_extra_1.default.existsSync(path_1.default.join(cwd, '.cursor'))) {
         return 'cursor';
@@ -19,12 +23,16 @@ function detectIDE(cwd) {
     if (fs_extra_1.default.existsSync(path_1.default.join(cwd, '.vscode'))) {
         return 'vscode';
     }
-    // Check environment variables that IDEs set
+    // Check environment variables that IDEs/Agents set
     const termProgram = process.env.TERM_PROGRAM || '';
     const terminal = process.env.TERMINAL_EMULATOR || '';
+    const appName = process.env.APP_NAME || '';
     if (termProgram.toLowerCase().includes('cursor') || terminal.toLowerCase().includes('cursor')) {
         return 'cursor';
     }
+    if (termProgram.toLowerCase().includes('cline') || appName.toLowerCase().includes('cline')) {
+        return 'cline';
+    }
     if (termProgram.toLowerCase().includes('vscode') || process.env.VSCODE_INJECTION) {
         return 'vscode';
     }
@@ -136,6 +144,7 @@ npx @rigour-labs/cli check
 npx @rigour-labs/cli run -- <agent-command>
 \`\`\`
 
+${constants_js_1.AGNOSTIC_AI_INSTRUCTIONS}
 ${constants_js_1.CODE_QUALITY_RULES}
 
 ${constants_js_1.DEBUGGING_RULES}
@@ -168,10 +177,18 @@ ${ruleContent}`;
             console.log(chalk_1.default.green('✔ Initialized Cursor Handshake (.cursor/rules/rigour.mdc)'));
         }
     }
-    if (targetIDE === 'vscode') {
+    if (targetIDE === 'vscode' || targetIDE === 'all') {
         // VS Code users use the universal AGENT_INSTRUCTIONS.md (already created above)
+        // We could also add .vscode/settings.json or snippets here if needed
         console.log(chalk_1.default.green('✔ VS Code mode - using Universal Handshake (docs/AGENT_INSTRUCTIONS.md)'));
     }
+    if (targetIDE === 'cline' || targetIDE === 'all') {
+        const clineRulesPath = path_1.default.join(cwd, '.clinerules');
+        if (!(await fs_extra_1.default.pathExists(clineRulesPath))) {
+            await fs_extra_1.default.writeFile(clineRulesPath, ruleContent);
+            console.log(chalk_1.default.green('✔ Initialized Cline Handshake (.clinerules)'));
+        }
+    }
     // 3. Update .gitignore
     const gitignorePath = path_1.default.join(cwd, '.gitignore');
     const ignorePatterns = ['rigour-report.json', 'rigour-fix-packet.json', '.rigour/'];

package/dist/init-rules.test.js

@@ -15,23 +15,29 @@ const path_1 = __importDefault(require("path"));
     (0, vitest_1.afterEach)(async () => {
         await fs_extra_1.default.remove(testDir);
     });
-    (0, vitest_1.it)('should create universal instructions and cursor rules on init', async () => {
-        // Run init in test directory
-        await (0, init_js_1.initCommand)(testDir);
+    (0, vitest_1.it)('should create instructions with agnostic rules and cursor rules on init', async () => {
+        // Run init in test directory with all IDEs to verify rules in both locations
+        await (0, init_js_1.initCommand)(testDir, { ide: 'all' });
         const instructionsPath = path_1.default.join(testDir, 'docs', 'AGENT_INSTRUCTIONS.md');
         const mdcPath = path_1.default.join(testDir, '.cursor', 'rules', 'rigour.mdc');
         (0, vitest_1.expect)(await fs_extra_1.default.pathExists(instructionsPath)).toBe(true);
         (0, vitest_1.expect)(await fs_extra_1.default.pathExists(mdcPath)).toBe(true);
         const instructionsContent = await fs_extra_1.default.readFile(instructionsPath, 'utf-8');
         const mdcContent = await fs_extra_1.default.readFile(mdcPath, 'utf-8');
+        // Check for agnostic instructions
+        (0, vitest_1.expect)(instructionsContent).toContain('# 🤖 CRITICAL INSTRUCTION FOR AI');
+        (0, vitest_1.expect)(instructionsContent).toContain('VERIFICATION PROOF REQUIRED');
         // Check for key sections in universal instructions
         (0, vitest_1.expect)(instructionsContent).toContain('# 🛡️ Rigour: Engineering Excellence Protocol');
         (0, vitest_1.expect)(instructionsContent).toContain('# Code Quality Standards');
-        (0, vitest_1.expect)(instructionsContent).toContain('# Investigation & Debugging Protocol');
-        (0, vitest_1.expect)(instructionsContent).toContain('# Role & Collaboration');
-        // Check that MDC includes frontmatter and same rules
-        (0, vitest_1.expect)(mdcContent).toContain('---');
-        (0, vitest_1.expect)(mdcContent).toContain('description: Enforcement of Rigour quality gates and best practices.');
-        (0, vitest_1.expect)(mdcContent).toContain('# Code Quality Standards');
+        // Check that MDC includes agnostic rules
+        (0, vitest_1.expect)(mdcContent).toContain('# 🤖 CRITICAL INSTRUCTION FOR AI');
+    });
+    (0, vitest_1.it)('should create .clinerules when ide is cline or all', async () => {
+        await (0, init_js_1.initCommand)(testDir, { ide: 'cline' });
+        const clineRulesPath = path_1.default.join(testDir, '.clinerules');
+        (0, vitest_1.expect)(await fs_extra_1.default.pathExists(clineRulesPath)).toBe(true);
+        const content = await fs_extra_1.default.readFile(clineRulesPath, 'utf-8');
+        (0, vitest_1.expect)(content).toContain('# 🤖 CRITICAL INSTRUCTION FOR AI');
     });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rigour-labs/cli",
-  "version": "2.7.0",
+  "version": "2.8.0",
   "bin": {
     "rigour": "dist/cli.js"
   },
@@ -21,7 +21,7 @@
     "globby": "^14.0.1",
     "inquirer": "9.2.16",
     "yaml": "^2.8.2",
-    "@rigour-labs/core": "2.7.0"
+    "@rigour-labs/core": "2.8.0"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",

package/src/commands/constants.ts

@@ -185,3 +185,25 @@ If thresholds genuinely need adjustment, escalate to the team lead with justific
 8. ❌ Losing context between messages
 9. ❌ Modifying rigour.yml to pass quality checks
 `;
+
+export const AGNOSTIC_AI_INSTRUCTIONS = `
+# 🤖 CRITICAL INSTRUCTION FOR AI
+
+When asked to implement a feature or fix an issue:
+
+1. **FIRST**: Read ALL relevant rules files (e.g., \`.cursor/rules/*.mdc\`, \`.clinerules\`, \`docs/AGENT_INSTRUCTIONS.md\`).
+2. **NEVER** claim "done" until you can demonstrate the feature WORKS or the fix is verified.
+3. **ALWAYS** run \`npx @rigour-labs/cli check\` before claiming completion.
+4. **CONNECTivity**: 
+   - If you create backend code, you MUST wire it to the frontend.
+   - If you create a service, you MUST connect it to the UI.
+
+## ✅ VERIFICATION PROOF REQUIRED
+
+Before saying "done", show ONE of:
+- Screenshot/output proving it works.
+- The exact commands the user can run to verify.
+- The full flow from initiation (e.g., button click) to result.
+
+"Code exists" ≠ "Feature works"
+`;

package/src/commands/init.ts

@@ -3,44 +3,55 @@ import path from 'path';
 import chalk from 'chalk';
 import yaml from 'yaml';
 import { DiscoveryService } from '@rigour-labs/core';
-import { CODE_QUALITY_RULES, DEBUGGING_RULES, COLLABORATION_RULES } from './constants.js';
+import { CODE_QUALITY_RULES, DEBUGGING_RULES, COLLABORATION_RULES, AGNOSTIC_AI_INSTRUCTIONS } from './constants.js';
 
 export interface InitOptions {
     preset?: string;
     paradigm?: string;
-    ide?: 'cursor' | 'vscode' | 'all';
+    ide?: 'cursor' | 'vscode' | 'cline' | 'all';
     dryRun?: boolean;
     explain?: boolean;
 }
 
-type DetectedIDE = 'cursor' | 'vscode' | 'unknown';
+type DetectedIDE = 'cursor' | 'vscode' | 'cline' | 'unknown';
 
 function detectIDE(cwd: string): DetectedIDE {
+    // Check for Cline-specific markers
+    if (fs.existsSync(path.join(cwd, '.clinerules'))) {
+        return 'cline';
+    }
+
     // Check for Cursor-specific markers
     if (fs.existsSync(path.join(cwd, '.cursor'))) {
         return 'cursor';
     }
-    
+
     // Check for VS Code markers
     if (fs.existsSync(path.join(cwd, '.vscode'))) {
         return 'vscode';
     }
-    
-    // Check environment variables that IDEs set
+
+    // Check environment variables that IDEs/Agents set
     const termProgram = process.env.TERM_PROGRAM || '';
     const terminal = process.env.TERMINAL_EMULATOR || '';
-    
+    const appName = process.env.APP_NAME || '';
+
     if (termProgram.toLowerCase().includes('cursor') || terminal.toLowerCase().includes('cursor')) {
         return 'cursor';
     }
-    
+
+    if (termProgram.toLowerCase().includes('cline') || appName.toLowerCase().includes('cline')) {
+        return 'cline';
+    }
+
     if (termProgram.toLowerCase().includes('vscode') || process.env.VSCODE_INJECTION) {
         return 'vscode';
     }
-    
+
     return 'unknown';
 }
 
+
 export async function initCommand(cwd: string, options: InitOptions = {}) {
     const discovery = new DiscoveryService();
     const result = await discovery.discover(cwd);
@@ -152,6 +163,7 @@ npx @rigour-labs/cli check
 npx @rigour-labs/cli run -- <agent-command>
 \`\`\`
 
+${AGNOSTIC_AI_INSTRUCTIONS}
 ${CODE_QUALITY_RULES}
 
 ${DEBUGGING_RULES}
@@ -168,11 +180,11 @@ ${COLLABORATION_RULES}
     // 2. Create IDE-Specific Rules based on detection or user preference
     const detectedIDE = detectIDE(cwd);
     const targetIDE = options.ide || (detectedIDE !== 'unknown' ? detectedIDE : 'all');
-    
+
     if (detectedIDE !== 'unknown' && !options.ide) {
         console.log(chalk.dim(`   (Auto-detected IDE: ${detectedIDE})`));
     }
-    
+
     if (targetIDE === 'cursor' || targetIDE === 'all') {
         const cursorRulesDir = path.join(cwd, '.cursor', 'rules');
         await fs.ensureDir(cursorRulesDir);
@@ -189,12 +201,21 @@ ${ruleContent}`;
             console.log(chalk.green('✔ Initialized Cursor Handshake (.cursor/rules/rigour.mdc)'));
         }
     }
-    
-    if (targetIDE === 'vscode') {
+
+    if (targetIDE === 'vscode' || targetIDE === 'all') {
         // VS Code users use the universal AGENT_INSTRUCTIONS.md (already created above)
+        // We could also add .vscode/settings.json or snippets here if needed
         console.log(chalk.green('✔ VS Code mode - using Universal Handshake (docs/AGENT_INSTRUCTIONS.md)'));
     }
 
+    if (targetIDE === 'cline' || targetIDE === 'all') {
+        const clineRulesPath = path.join(cwd, '.clinerules');
+        if (!(await fs.pathExists(clineRulesPath))) {
+            await fs.writeFile(clineRulesPath, ruleContent);
+            console.log(chalk.green('✔ Initialized Cline Handshake (.clinerules)'));
+        }
+    }
+
     // 3. Update .gitignore
     const gitignorePath = path.join(cwd, '.gitignore');
     const ignorePatterns = ['rigour-report.json', 'rigour-fix-packet.json', '.rigour/'];

package/src/init-rules.test.ts

@@ -14,9 +14,9 @@ describe('Init Command Rules Verification', () => {
         await fs.remove(testDir);
     });
 
-    it('should create universal instructions and cursor rules on init', async () => {
-        // Run init in test directory
-        await initCommand(testDir);
+    it('should create instructions with agnostic rules and cursor rules on init', async () => {
+        // Run init in test directory with all IDEs to verify rules in both locations
+        await initCommand(testDir, { ide: 'all' });
 
         const instructionsPath = path.join(testDir, 'docs', 'AGENT_INSTRUCTIONS.md');
         const mdcPath = path.join(testDir, '.cursor', 'rules', 'rigour.mdc');
@@ -27,15 +27,25 @@ describe('Init Command Rules Verification', () => {
         const instructionsContent = await fs.readFile(instructionsPath, 'utf-8');
         const mdcContent = await fs.readFile(mdcPath, 'utf-8');
 
+        // Check for agnostic instructions
+        expect(instructionsContent).toContain('# 🤖 CRITICAL INSTRUCTION FOR AI');
+        expect(instructionsContent).toContain('VERIFICATION PROOF REQUIRED');
+
         // Check for key sections in universal instructions
         expect(instructionsContent).toContain('# 🛡️ Rigour: Engineering Excellence Protocol');
         expect(instructionsContent).toContain('# Code Quality Standards');
-        expect(instructionsContent).toContain('# Investigation & Debugging Protocol');
-        expect(instructionsContent).toContain('# Role & Collaboration');
 
-        // Check that MDC includes frontmatter and same rules
-        expect(mdcContent).toContain('---');
-        expect(mdcContent).toContain('description: Enforcement of Rigour quality gates and best practices.');
-        expect(mdcContent).toContain('# Code Quality Standards');
+        // Check that MDC includes agnostic rules
+        expect(mdcContent).toContain('# 🤖 CRITICAL INSTRUCTION FOR AI');
     });
+
+    it('should create .clinerules when ide is cline or all', async () => {
+        await initCommand(testDir, { ide: 'cline' });
+        const clineRulesPath = path.join(testDir, '.clinerules');
+        expect(await fs.pathExists(clineRulesPath)).toBe(true);
+
+        const content = await fs.readFile(clineRulesPath, 'utf-8');
+        expect(content).toContain('# 🤖 CRITICAL INSTRUCTION FOR AI');
+    });
+
 });