principles-disciple 1.73.0 → 1.74.0

package/templates/langs/en/skills/plan-script/SKILL.md

@@ -1,32 +0,0 @@
----
-name: plan-script
-description: Create a step-by-step movie-script style execution plan. Includes target files, verification metrics, and rollback strategy.
-disable-model-invocation: true
----
-
-# Plan Script
-
-**Goal**: Produce a "foolproof" executable plan to ensure controlled execution.
-
-Please generate plan in the following structure:
-
-## 1. Target Files (Authorization List)
-- List file paths **uniquely authorized** for modification in this plan.
-- Format: `- path/to/file`
-
-## 2. Steps (Execution Steps)
-1. Operations specific to filenames and tool calls.
-2. Each step includes expected intermediate state.
-
-## 3. Metrics (Verification Metrics)
-- How to quantitatively prove this plan succeeded? (e.g., tests pass, command returns 0, specific string appears in logs).
-
-## 4. Active Mental Models
-- Select exactly **2** meta-cognitive models from `.principles/THINKING_OS.md` that are most relevant to the current task.
-- Format: `- [T-0X] Model Name: Why is it needed for this specific task?`
-
-## 5. Rollback (Rollback Strategy)
-- If step 2 fails, how to one-click restore to safe state?
-
----
-**Action**: Update above content to `PLAN.md` and set `STATUS: READY`.

package/templates/langs/zh/skills/plan-script/SKILL.md

@@ -1,32 +0,0 @@
----
-name: plan-script
-description: Create a step-by-step movie-script style execution plan. Includes target files, verification metrics, and rollback strategy.
-disable-model-invocation: true
----
-
-# Plan Script (计划编排)
-
-**目标**: 产生一份“傻瓜式”可执行计划，确保执行过程受控。
-
-请按以下结构生成计划：
-
-## 1. Target Files (授权清单)
-- 列出本次计划**唯一授权**修改的文件路径。
-- 格式：`- path/to/file`
-
-## 2. Steps (执行步骤)
-1. 具体到文件名和工具调用的操作。
-2. 每个步骤包含预期的中间状态。
-
-## 3. Metrics (验证指标)
-- 如何量化证明本计划成功了？(如：测试通过、命令返回 0、日志出现特定字符串)。
-
-## 4. Active Mental Models (激活的思维模型)
-- 从 `.principles/THINKING_OS.md` 中挑选 **2 个** 最适合当前任务的元认知模型。
-- 格式：`- [T-0X] 模型名称：为什么在这个任务中需要它？`
-
-## 5. Rollback (回滚方案)
-- 如果步骤 2 失败，如何一键恢复到安全状态？
-
----
-**动作**: 请将以上内容更新至 `PLAN.md`，并设置 `STATUS: READY`。

package/tests/hooks/confirm-first-gate.test.ts

@@ -1,333 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import * as os from 'os';
-import * as path from 'path';
-import * as fs from 'fs';
-import {
-  evaluateConfirmFirstGateSync,
-  detectApprovalMarker,
-  setConfirmFirstDirective,
-  setConfirmFirstApproval,
-  resetConfirmFirst,
-  isSessionApproved,
-  hasActiveDirective,
-  clearAllConfirmFirstState,
-  setConfirmFirstStore,
-  hydrateFromStore,
-} from '../../src/core/confirm-first-gate.js';
-import { SqliteConnection } from '@principles/core/runtime-v2';
-import { SqliteConfirmFirstStateStore } from '@principles/core/runtime-v2';
-
-describe('Confirm-First Gate', () => {
-  beforeEach(() => {
-    clearAllConfirmFirstState();
-  });
-
-  describe('detectApprovalMarker', () => {
-    it('detects Chinese approval markers', () => {
-      expect(detectApprovalMarker('确认')).toBe(true);
-      expect(detectApprovalMarker('批准')).toBe(true);
-      expect(detectApprovalMarker('按计划执行')).toBe(true);
-      expect(detectApprovalMarker('可以执行')).toBe(true);
-      expect(detectApprovalMarker('就这么做')).toBe(true);
-      expect(detectApprovalMarker('去执行')).toBe(true);
-      expect(detectApprovalMarker('开始执行')).toBe(true);
-      expect(detectApprovalMarker('执行吧')).toBe(true);
-      expect(detectApprovalMarker('同意')).toBe(true);
-    });
-
-    it('detects English approval markers', () => {
-      expect(detectApprovalMarker('approved')).toBe(true);
-      expect(detectApprovalMarker('go ahead')).toBe(true);
-      expect(detectApprovalMarker('lgtm')).toBe(true);
-      expect(detectApprovalMarker('yes, do it')).toBe(true);
-      expect(detectApprovalMarker('do it')).toBe(true);
-      expect(detectApprovalMarker('yes, proceed')).toBe(true);
-      expect(detectApprovalMarker('yes, execute')).toBe(true);
-      expect(detectApprovalMarker('proceed with the plan')).toBe(true);
-      expect(detectApprovalMarker('execute the plan')).toBe(true);
-      expect(detectApprovalMarker('please proceed with the plan')).toBe(true);
-    });
-
-    it('rejects vague text', () => {
-      expect(detectApprovalMarker('看看')).toBe(false);
-      expect(detectApprovalMarker('继续想想')).toBe(false);
-      expect(detectApprovalMarker('你决定')).toBe(false);
-      expect(detectApprovalMarker('hello world')).toBe(false);
-      expect(detectApprovalMarker('')).toBe(false);
-    });
-
-    it('rejects negated Chinese approval', () => {
-      expect(detectApprovalMarker('不同意')).toBe(false);
-      expect(detectApprovalMarker('不确认')).toBe(false);
-      expect(detectApprovalMarker('先不执行')).toBe(false);
-      expect(detectApprovalMarker('还没准备好确认')).toBe(false);
-      expect(detectApprovalMarker('暂不批准')).toBe(false);
-    });
-
-    it('rejects negated English approval', () => {
-      expect(detectApprovalMarker("don't proceed")).toBe(false);
-      expect(detectApprovalMarker("don't do it")).toBe(false);
-      expect(detectApprovalMarker("not ready to confirm")).toBe(false);
-      expect(detectApprovalMarker("can't approve yet")).toBe(false);
-      expect(detectApprovalMarker("won't proceed")).toBe(false);
-      expect(detectApprovalMarker("stop")).toBe(false);
-    });
-
-    it('rejects ambiguous English phrases without explicit approval context', () => {
-      expect(detectApprovalMarker('please confirm requirements before proceeding')).toBe(false);
-      expect(detectApprovalMarker('how should we proceed?')).toBe(false);
-      expect(detectApprovalMarker('confirm the requirement first')).toBe(false);
-      expect(detectApprovalMarker('should I proceed?')).toBe(false);
-      expect(detectApprovalMarker('I need to confirm something')).toBe(false);
-      expect(detectApprovalMarker('let me confirm the plan')).toBe(false);
-    });
-  });
-
-  describe('evaluateConfirmFirstGateSync', () => {
-    it('skips when no sessionId', () => {
-      const result = evaluateConfirmFirstGateSync(undefined, 'write', {});
-      expect(result.action).toBe('skip');
-    });
-
-    it('skips when no confirm-first directive active', () => {
-      const result = evaluateConfirmFirstGateSync('session-1', 'write', {});
-      expect(result.action).toBe('skip');
-    });
-
-    it('allows non-mutating tools even with active directive', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'read', {});
-      expect(result.action).toBe('allow');
-    });
-
-    it('blocks write tool when directive active and not approved', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'write', { path: 'test.json' });
-      expect(result.action).toBe('block');
-      expect(result.reason).toBe('confirm_first_required');
-      expect(result.principleId).toBe('princ-mvp-acceptance-confirm-first');
-      expect(result.nextAction).toContain('owner approval');
-    });
-
-    it('blocks edit tool when directive active and not approved', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'edit', { file_path: 'test.ts' });
-      expect(result.action).toBe('block');
-      expect(result.reason).toBe('confirm_first_required');
-    });
-
-    it('blocks delete_file when directive active and not approved', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'delete_file', { path: 'test.txt' });
-      expect(result.action).toBe('block');
-    });
-
-    it('blocks mutating exec when directive active and not approved', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'exec', { command: 'rm -rf /tmp/test' });
-      expect(result.action).toBe('block');
-    });
-
-    it('allows non-mutating exec when directive active and not approved', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'exec', { command: 'ls -la' });
-      expect(result.action).toBe('allow');
-    });
-
-    it('allows bash with undefined params when directive active', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'bash', undefined);
-      expect(result.action).toBe('allow');
-    });
-
-    it('allows read tool when directive active and not approved', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'read', { file_path: 'test.ts' });
-      expect(result.action).toBe('allow');
-    });
-
-    it('allows write after approval', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      setConfirmFirstApproval('session-1');
-      const result = evaluateConfirmFirstGateSync('session-1', 'write', { path: 'test.json' });
-      expect(result.action).toBe('allow');
-      expect(isSessionApproved('session-1')).toBe(true);
-    });
-
-    it('approval is session-scoped', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      setConfirmFirstDirective('session-2', true, 'princ-mvp-acceptance-confirm-first');
-      setConfirmFirstApproval('session-1');
-
-      expect(evaluateConfirmFirstGateSync('session-1', 'write', {}).action).toBe('allow');
-      expect(evaluateConfirmFirstGateSync('session-2', 'write', {}).action).toBe('block');
-    });
-
-    it('blocks apply_patch with no path when directive active', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      const result = evaluateConfirmFirstGateSync('session-1', 'apply_patch', { patch: '@@ -1 +1 @@\n-old\n+new' });
-      expect(result.action).toBe('block');
-      expect(result.reason).toBe('confirm_first_required');
-    });
-
-    it('allows apply_patch after approval', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      setConfirmFirstApproval('session-1');
-      const result = evaluateConfirmFirstGateSync('session-1', 'apply_patch', { patch: '@@ -1 +1 @@\n-old\n+new' });
-      expect(result.action).toBe('allow');
-    });
-
-    it('reset clears both directive and approval state', () => {
-      setConfirmFirstDirective('session-1', true, 'princ-mvp-acceptance-confirm-first');
-      setConfirmFirstApproval('session-1');
-      resetConfirmFirst('session-1');
-
-      expect(hasActiveDirective('session-1')).toBe(false);
-      expect(isSessionApproved('session-1')).toBe(false);
-      expect(evaluateConfirmFirstGateSync('session-1', 'write', {}).action).toBe('skip');
-    });
-  });
-});
-
-describe('Cross-restart persistence', () => {
-  let tmpDir: string;
-  let connection: SqliteConnection;
-  let store: SqliteConfirmFirstStateStore;
-
-  beforeEach(() => {
-    clearAllConfirmFirstState();
-    setConfirmFirstStore(null);
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pd-cf-test-'));
-    connection = new SqliteConnection(tmpDir);
-    store = new SqliteConfirmFirstStateStore(connection);
-  });
-
-  afterEach(() => {
-    setConfirmFirstStore(null);
-    clearAllConfirmFirstState();
-    try {
-      connection.close();
-    } catch {}
-    try {
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    } catch {}
-  });
-
-  it('directive + approval survive restart', () => {
-    setConfirmFirstStore(store);
-    setConfirmFirstDirective('sess-restart', true, 'princ-123');
-    setConfirmFirstApproval('sess-restart');
-
-    expect(evaluateConfirmFirstGateSync('sess-restart', 'write', {}).action).toBe('allow');
-
-    setConfirmFirstStore(null);
-    clearAllConfirmFirstState();
-    setConfirmFirstStore(store);
-    hydrateFromStore('sess-restart');
-
-    expect(evaluateConfirmFirstGateSync('sess-restart', 'write', {}).action).toBe('allow');
-    expect(hasActiveDirective('sess-restart')).toBe(true);
-    expect(isSessionApproved('sess-restart')).toBe(true);
-  });
-
-  it('directive without approval survives restart', () => {
-    setConfirmFirstStore(store);
-    setConfirmFirstDirective('sess-restart', true, 'princ-456');
-
-    expect(evaluateConfirmFirstGateSync('sess-restart', 'write', {}).action).toBe('block');
-
-    setConfirmFirstStore(null);
-    clearAllConfirmFirstState();
-    setConfirmFirstStore(store);
-    hydrateFromStore('sess-restart');
-
-    expect(evaluateConfirmFirstGateSync('sess-restart', 'write', {}).action).toBe('block');
-    expect(hasActiveDirective('sess-restart')).toBe(true);
-    expect(isSessionApproved('sess-restart')).toBe(false);
-  });
-
-  it('no directive survives restart', () => {
-    setConfirmFirstStore(store);
-
-    setConfirmFirstStore(null);
-    clearAllConfirmFirstState();
-    setConfirmFirstStore(store);
-    hydrateFromStore('sess-noexist');
-
-    expect(evaluateConfirmFirstGateSync('sess-noexist', 'write', {}).action).toBe('skip');
-    expect(hasActiveDirective('sess-noexist')).toBe(false);
-  });
-});
-
-describe('Store degradation (ERR-002)', () => {
-  afterEach(() => {
-    setConfirmFirstStore(null);
-    clearAllConfirmFirstState();
-  });
-
-  it('store write failure degrades gracefully to cache-only', () => {
-    const throwingStore = {
-      upsertDirective: () => { throw new Error('DB unavailable'); },
-      upsertApproval: () => { throw new Error('DB unavailable'); },
-      getState: () => null,
-      deleteState: () => { throw new Error('DB unavailable'); },
-      deleteAllState: () => { throw new Error('DB unavailable'); },
-      pruneStaleRows: () => 0,
-      getAllState: () => [],
-    } as unknown as SqliteConfirmFirstStateStore;
-
-    setConfirmFirstStore(throwingStore);
-    setConfirmFirstDirective('sess-degrade', true, 'princ-123');
-
-    expect(hasActiveDirective('sess-degrade')).toBe(true);
-
-    setConfirmFirstApproval('sess-degrade');
-    expect(isSessionApproved('sess-degrade')).toBe(true);
-    expect(evaluateConfirmFirstGateSync('sess-degrade', 'write', {}).action).toBe('allow');
-  });
-});
-
-describe('Stale directive cleared on reset (PRI-266)', () => {
-  beforeEach(() => {
-    clearAllConfirmFirstState();
-  });
-
-  it('resetConfirmFirst clears directive and approval from cache and store', () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pd-cf-stale-'));
-    try {
-      const connection = new SqliteConnection(tmpDir);
-      const store = new SqliteConfirmFirstStateStore(connection);
-      setConfirmFirstStore(store);
-
-      setConfirmFirstDirective('sess-stale', true, 'princ-stale');
-      setConfirmFirstApproval('sess-stale');
-
-      expect(hasActiveDirective('sess-stale')).toBe(true);
-      expect(isSessionApproved('sess-stale')).toBe(true);
-
-      resetConfirmFirst('sess-stale');
-
-      expect(hasActiveDirective('sess-stale')).toBe(false);
-      expect(isSessionApproved('sess-stale')).toBe(false);
-      expect(evaluateConfirmFirstGateSync('sess-stale', 'write', {}).action).toBe('skip');
-
-      connection.close();
-    } finally {
-      setConfirmFirstStore(null);
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    }
-  });
-
-  it('resetConfirmFirst without store clears in-memory cache only', () => {
-    setConfirmFirstDirective('sess-nostore', true, 'princ-nostore');
-    setConfirmFirstApproval('sess-nostore');
-
-    expect(hasActiveDirective('sess-nostore')).toBe(true);
-    expect(isSessionApproved('sess-nostore')).toBe(true);
-
-    resetConfirmFirst('sess-nostore');
-
-    expect(hasActiveDirective('sess-nostore')).toBe(false);
-    expect(isSessionApproved('sess-nostore')).toBe(false);
-  });
-});