oh-my-copilot 4.13.2 → 4.13.21

package/scripts/post-tool-verifier.mjs

@@ -747,11 +747,116 @@ function extractTextFromKnownToolResponseField(value, depth = 0) {
   return texts;
 }
 
-function hasStructuredWriteSuccess(rawResponse) {
+function isStructuredEnvelopePayloadField(fieldName) {
+  return new Set([
+    'content',
+    'oldstring',
+    'newstring',
+    'originalfile',
+    'structuredpatch',
+    'patch',
+    'diff',
+    'lines',
+    'line',
+  ]).has(fieldName.toLowerCase());
+}
+
+function isStructuredEnvelopeStatusTextField(fieldName) {
+  return new Set(['message', 'output', 'stdout', 'stderr', 'text', 'result'])
+    .has(fieldName.toLowerCase());
+}
+
+function hasExplicitStructuredFailureIndicator(value, depth = 0, fieldName = '') {
+  if (!value || typeof value === 'string' || depth > 4) return false;
+  if (fieldName && isStructuredEnvelopePayloadField(fieldName)) return false;
+
+  if (Array.isArray(value)) {
+    return value.some(item => hasExplicitStructuredFailureIndicator(item, depth + 1, fieldName));
+  }
+
+  if (typeof value !== 'object') return false;
+
+  for (const [key, fieldValue] of Object.entries(value)) {
+    const normalizedKey = key.toLowerCase();
+    if (
+      typeof fieldValue === 'string' &&
+      isStructuredEnvelopeStatusTextField(key) &&
+      detectWriteFailure(fieldValue)
+    ) {
+      return true;
+    }
+
+    if (
+      (normalizedKey.includes('error') || normalizedKey.includes('fail')) &&
+      fieldValue !== false &&
+      fieldValue !== 0 &&
+      fieldValue !== null &&
+      fieldValue !== undefined &&
+      !(typeof fieldValue === 'string' && fieldValue.trim() === '') &&
+      !(Array.isArray(fieldValue) && fieldValue.length === 0)
+    ) {
+      return true;
+    }
+
+    if (hasExplicitStructuredFailureIndicator(fieldValue, depth + 1, key)) {
+      return true;
+    }
+  }
+
+  return false;
+}
+
+function hasEditEnvelopeSuccess(value, depth = 0) {
+  if (!value || typeof value === 'string' || depth > 4) return false;
+
+  if (Array.isArray(value)) {
+    return value.some(item => hasEditEnvelopeSuccess(item, depth + 1));
+  }
+
+  if (typeof value !== 'object') return false;
+
+  if (hasExplicitStructuredFailureIndicator(value, depth)) return false;
+
+  if (typeof value.filePath === 'string' && Array.isArray(value.structuredPatch)) {
+    return true;
+  }
+
+  return Object.values(value).some(item => hasEditEnvelopeSuccess(item, depth + 1));
+}
+
+function hasWriteEnvelopeSuccess(value, depth = 0) {
+  if (!value || typeof value === 'string' || depth > 4) return false;
+
+  if (Array.isArray(value)) {
+    return value.some(item => hasWriteEnvelopeSuccess(item, depth + 1));
+  }
+
+  if (typeof value !== 'object') return false;
+
+  if (hasExplicitStructuredFailureIndicator(value, depth)) return false;
+
+  if (
+    typeof value.filePath === 'string' &&
+    (value.type === 'create' || value.type === 'update')
+  ) {
+    return true;
+  }
+
+  return Object.values(value).some(item => hasWriteEnvelopeSuccess(item, depth + 1));
+}
+
+function hasStructuredWriteSuccess(rawResponse, toolName = '') {
   if (!rawResponse || typeof rawResponse === 'string') return false;
+  if (toolName === 'Edit' && hasEditEnvelopeSuccess(rawResponse)) return true;
+  if (toolName === 'Write' && hasWriteEnvelopeSuccess(rawResponse)) return true;
   return extractTextFromKnownToolResponseField(rawResponse).some(isCopilotCliWriteSuccess);
 }
 
+function hasStructuredWriteFailure(rawResponse) {
+  if (!rawResponse || typeof rawResponse === 'string') return false;
+  return hasExplicitStructuredFailureIndicator(rawResponse);
+}
+
 // Get agent completion summary from tracking state
 function getAgentCompletionSummary(directory, quietLevel = QUIET_LEVEL) {
   const trackingFile = join(directory, '.omcp', 'state', 'subagent-tracking.json');
@@ -780,7 +885,12 @@ function getAgentCompletionSummary(directory, quietLevel = QUIET_LEVEL) {
 
 // Generate contextual message
 function generateMessage(toolName, toolOutput, sessionId, toolCount, directory, options = {}) {
-  const { wasTruncated = false, rawLength = 0, structuredWriteSuccess = false } = options;
+  const {
+    wasTruncated = false,
+    rawLength = 0,
+    structuredWriteSuccess = false,
+    structuredWriteFailure = false,
+  } = options;
   let message = '';
 
   switch (toolName) {
@@ -831,7 +941,7 @@ function generateMessage(toolName, toolOutput, sessionId, toolCount, directory,
     }
 
     case 'Edit':
-      if (!structuredWriteSuccess && !isCopilotCliWriteSuccess(toolOutput) && detectWriteFailure(toolOutput)) {
+      if (structuredWriteFailure || (!structuredWriteSuccess && !isCopilotCliWriteSuccess(toolOutput) && detectWriteFailure(toolOutput))) {
         message = 'Edit operation failed. Verify file exists and content matches exactly.';
       } else if (QUIET_LEVEL === 0) {
         message = 'Code modified. Verify changes work as expected before marking complete.';
@@ -839,7 +949,7 @@ function generateMessage(toolName, toolOutput, sessionId, toolCount, directory,
       break;
 
     case 'Write':
-      if (!structuredWriteSuccess && !isCopilotCliWriteSuccess(toolOutput) && detectWriteFailure(toolOutput)) {
+      if (structuredWriteFailure || (!structuredWriteSuccess && !isCopilotCliWriteSuccess(toolOutput) && detectWriteFailure(toolOutput))) {
         message = 'Write operation failed. Check file permissions and directory existence.';
       } else if (QUIET_LEVEL === 0) {
         message = 'File written. Test the changes to ensure they work correctly.';
@@ -897,7 +1007,9 @@ async function main() {
     const toolName = data.tool_name || data.toolName || '';
     const rawResponse = data.tool_response || data.toolOutput || '';
     const structuredWriteSuccess =
-      (toolName === 'Write' || toolName === 'Edit') && hasStructuredWriteSuccess(rawResponse);
+      (toolName === 'Write' || toolName === 'Edit') && hasStructuredWriteSuccess(rawResponse, toolName);
+    const structuredWriteFailure =
+      (toolName === 'Write' || toolName === 'Edit') && hasStructuredWriteFailure(rawResponse);
     const toolOutput = typeof rawResponse === 'string' ? rawResponse : JSON.stringify(rawResponse);
     const { clipped: clippedToolOutput, wasTruncated } = clipToolOutputForAnalysis(toolName, toolOutput);
     const sessionId = data.session_id || data.sessionId || 'unknown';
@@ -944,6 +1056,7 @@ async function main() {
         wasTruncated,
         rawLength: toolOutput.length,
         structuredWriteSuccess,
+        structuredWriteFailure,
       }),
       maybeBuildPreemptiveCompactionMessage(toolName, data, directory),
     );

package/scripts/pre-tool-enforcer.mjs

@@ -10,7 +10,7 @@ import { existsSync, mkdirSync, readFileSync, readdirSync, renameSync, writeFile
 import { dirname, join, resolve } from 'path';
 import { execSync } from 'child_process';
 import { fileURLToPath, pathToFileURL } from 'url';
-import { getClaudeConfigDir } from './lib/config-dir.mjs';
+import { getCopilotConfigDir } from './lib/config-dir.mjs';
 import { evaluateAgentHeavyPreflight } from './lib/pre-tool-enforcer-preflight.mjs';
 import { resolveOmcStateRoot } from './lib/state-root.mjs';
 import { readStdin } from './lib/stdin.mjs';
@@ -29,6 +29,53 @@ function hasExtendedContextSuffix(modelId) {
 function isSubagentSafeModelId(modelId) {
   return isProviderSpecificModelId(modelId) && !hasExtendedContextSuffix(modelId);
 }
+function isBedrockProviderEnv() {
+  if (process.env.CLAUDE_CODE_USE_BEDROCK === '1') return true;
+  const modelId = process.env.CLAUDE_MODEL || process.env.ANTHROPIC_MODEL || '';
+  if (/^((us|eu|ap|global)\.anthropic\.|anthropic\.claude)/i.test(modelId)) return true;
+  if (
+    /^arn:aws(-[^:]+)?:bedrock:/i.test(modelId)
+    && /:(inference-profile|application-inference-profile)\//i.test(modelId)
+    && modelId.toLowerCase().includes('claude')
+  ) {
+    return true;
+  }
+  return false;
+}
+function isVertexProviderEnv() {
+  if (process.env.CLAUDE_CODE_USE_VERTEX === '1') return true;
+  const modelId = process.env.CLAUDE_MODEL || process.env.ANTHROPIC_MODEL || '';
+  return !!modelId && modelId.toLowerCase().startsWith('vertex_ai/');
+}
+function getActiveModelIds() {
+  return [process.env.CLAUDE_MODEL || '', process.env.ANTHROPIC_MODEL || ''].filter(Boolean);
+}
+function isNormalClaudeModelId(modelId) {
+  const lower = (modelId || '').toLowerCase();
+  return Boolean(lower) && lower.includes('claude') && !isProviderSpecificModelId(modelId);
+}
+function hasNormalClaudeActiveModel() {
+  return getActiveModelIds().some(isNormalClaudeModelId);
+}
+function isConfigForceInheritProxyEnv() {
+  const config = loadOmcConfig();
+  return config.routing?.forceInherit === true && !hasNormalClaudeActiveModel();
+}
+function isNonClaudeProviderEnv() {
+  if (isBedrockProviderEnv() || isVertexProviderEnv()) return true;
+  const modelId = process.env.CLAUDE_MODEL || process.env.ANTHROPIC_MODEL || '';
+  if (modelId && !modelId.toLowerCase().includes('claude')) return true;
+  const baseUrl = process.env.ANTHROPIC_BASE_URL || '';
+  if (baseUrl && !baseUrl.includes('anthropic.com')) return true;
+  return isConfigForceInheritProxyEnv();
+}
+function acceptsProxyAnthropicDefaultTierValue(key, value) {
+  return key.startsWith('ANTHROPIC_DEFAULT_')
+    && Boolean(value)
+    && isNonClaudeProviderEnv()
+    && !isBedrockProviderEnv()
+    && !isVertexProviderEnv();
+}
 const TIER_ALIASES = new Set(['sonnet', 'opus', 'haiku']);
 function isTierAlias(modelId) {
   return TIER_ALIASES.has((modelId || '').toLowerCase());
@@ -55,9 +102,10 @@ function resolveTierAliasToSafeModel(tierAlias) {
     // model resolution, which handles [1m] suffixes correctly for explicit model= calls.
     // OMC-internal vars (OMC_SUBAGENT_MODEL, OMC_MODEL_*) are not read by CC, so a [1m]
     // value there is not a valid routing proof — keep the stricter isSubagentSafeModelId check.
-    const isNativeCcVar = key.startsWith('ANTHROPIC_DEFAULT_') || key.startsWith('CLAUDE_CODE_BEDROCK_');
+    const isAnthropicDefaultTierVar = key.startsWith('ANTHROPIC_DEFAULT_');
+    const isNativeCcVar = isAnthropicDefaultTierVar || key.startsWith('CLAUDE_CODE_BEDROCK_');
     const validator = isNativeCcVar ? isProviderSpecificModelId : isSubagentSafeModelId;
-    if (value && validator(value)) return value;
+    if (value && (validator(value) || acceptsProxyAnthropicDefaultTierValue(key, value))) return value;
   }
   return '';
 }
@@ -167,7 +215,7 @@ function resolveTranscriptPath(transcriptPath, cwd) {
       const lastSep = transcriptPath.lastIndexOf('/');
       const sessionFile = lastSep !== -1 ? transcriptPath.substring(lastSep + 1) : '';
       if (sessionFile) {
-        const configDir = getClaudeConfigDir();
+        const configDir = getCopilotConfigDir();
         const projectsDir = join(configDir, 'projects');
         if (existsSync(projectsDir)) {
           const encodedMain = mainRepoRoot.replace(/[/\\]/g, '-');
@@ -518,8 +566,8 @@ function getSkillProtectionLevel(skillName, rawSkillName) {
 // Load OMC config to check forceInherit setting (issues #1135, #1201)
 function loadOmcConfig() {
   const configPaths = [
-    join(getClaudeConfigDir(), '.omc-config.json'),
-    join(process.cwd(), '.omc', 'config.json'),
+    join(getCopilotConfigDir(), '.omc-config.json'),
+    join(process.cwd(), '.omcp', 'config.json'),
   ];
   for (const configPath of configPaths) {
     try {

package/scripts/session-start.mjs

@@ -48,12 +48,10 @@ function readJsonFile(path) {
   }
 }
 
-
-const SESSION_ID_ALLOWLIST = /^[a-zA-Z0-9][a-zA-Z0-9_-]{0,255}$/;
 const WORKFLOW_SLOT_TOMBSTONE_TTL_MS = 24 * 60 * 60 * 1000;
 
 function isWorkflowSlotTombstonedForMode(omcRoot, mode, sessionId) {
-  const safeSessionId = typeof sessionId === 'string' && SESSION_ID_ALLOWLIST.test(sessionId) ? sessionId : '';
+  const safeSessionId = typeof sessionId === 'string' && SAFE_SESSION_ID_PATTERN.test(sessionId) ? sessionId : '';
   const ledgerPath = safeSessionId
     ? join(omcRoot, 'state', 'sessions', safeSessionId, 'skill-active-state.json')
     : join(omcRoot, 'state', 'skill-active-state.json');
@@ -72,6 +70,158 @@ function shouldRestoreModeState(omcRoot, mode, state, sessionId) {
   return true;
 }
 
+function readLinuxBootId() {
+  try {
+    if (!existsSync(LINUX_BOOT_ID_PATH)) return undefined;
+    const bootId = readFileSync(LINUX_BOOT_ID_PATH, 'utf-8').trim();
+    return bootId || undefined;
+  } catch {
+    return undefined;
+  }
+}
+
+function sessionStateDir(omcRoot, sessionId) {
+  return join(omcRoot, 'state', 'sessions', sessionId);
+}
+
+function sessionStartedMarkerPath(omcRoot, sessionId) {
+  return join(sessionStateDir(omcRoot, sessionId), SESSION_STARTED_MARKER_FILE);
+}
+
+function writeSessionStartedMarker(omcRoot, directory, sessionId) {
+  if (!sessionId || !SAFE_SESSION_ID_PATTERN.test(sessionId)) return;
+  try {
+    const dir = sessionStateDir(omcRoot, sessionId);
+    mkdirSync(dir, { recursive: true });
+    writeFileSync(
+      sessionStartedMarkerPath(omcRoot, sessionId),
+      JSON.stringify({
+        session_id: sessionId,
+        started_at: new Date().toISOString(),
+        cwd: directory,
+        pid: process.pid,
+        // Do not persist process.ppid here: installed hooks run through
+        // scripts/run.cjs, whose short-lived process exits as soon as this
+        // hook returns. Treating that runner PID as owner liveness caused
+        // later SessionStart hooks to falsely clean live session state.
+        boot_id: readLinuxBootId(),
+      }, null, 2),
+      { encoding: 'utf-8', mode: 0o600 },
+    );
+  } catch {
+    // Best-effort only; SessionStart must remain non-blocking.
+  }
+}
+
+function removeSessionStartedMarker(omcRoot, sessionId) {
+  if (!sessionId || !SAFE_SESSION_ID_PATTERN.test(sessionId)) return;
+  try {
+    const markerPath = sessionStartedMarkerPath(omcRoot, sessionId);
+    if (existsSync(markerPath)) unlinkSync(markerPath);
+  } catch {
+    // Best-effort only.
+  }
+}
+
+/**
+ * Return true only when SessionStart has durable abandonment evidence.
+ *
+ * Claude Code SessionStart input currently provides session metadata such as
+ * session_id, transcript_path, cwd, source, model, and agent_type, but no
+ * stable owner process for the interactive session. In installed OMC hooks the
+ * immediate hook parent belongs to scripts/run.cjs and is intentionally
+ * short-lived, so same-boot PID liveness checks are not reliable here. SessionEnd
+ * remains the primary same-boot cleanup path; SessionStart only reconciles
+ * durable leftovers, such as markers from a previous OS boot.
+ */
+function hasDurableAbandonmentEvidence(marker) {
+  const storedBootId = typeof marker?.boot_id === 'string' ? marker.boot_id : undefined;
+  const currentBootId = readLinuxBootId();
+  if (storedBootId && currentBootId && storedBootId !== currentBootId) {
+    return true;
+  }
+
+  // Same-boot hard-kill cleanup requires a durable owner signal. Claude Code
+  // does not currently provide one to hooks, so keep active state rather than
+  // guessing from hook-runner process ancestry or transcript metadata.
+  return false;
+}
+
+function cleanupSessionModeState(omcRoot, sessionId) {
+  const sessionDir = sessionStateDir(omcRoot, sessionId);
+  for (const file of SESSION_END_MODE_STATE_FILES) {
+    try {
+      const filePath = join(sessionDir, file);
+      const state = readJsonFile(filePath);
+      if (state?.active === true || file === 'skill-active-state.json') {
+        unlinkSync(filePath);
+      }
+    } catch {
+      // Leave ambiguous/unreadable state untouched.
+    }
+  }
+}
+
+function cleanupMissionStateForSession(omcRoot, sessionId) {
+  const missionStatePath = join(omcRoot, 'state', 'mission-state.json');
+  const parsed = readJsonFile(missionStatePath);
+  if (!Array.isArray(parsed?.missions)) return;
+
+  const before = parsed.missions.length;
+  parsed.missions = parsed.missions.filter((mission) => {
+    if (mission?.source !== 'session') return true;
+    const missionId = typeof mission.id === 'string' ? mission.id : '';
+    return !missionId.includes(sessionId);
+  });
+  if (parsed.missions.length !== before) {
+    parsed.updatedAt = new Date().toISOString();
+    try {
+      writeFileSync(missionStatePath, JSON.stringify(parsed, null, 2));
+    } catch {
+      // Best-effort only.
+    }
+  }
+}
+
+function reconcileAbandonedSessionStarts(omcRoot, currentSessionId) {
+  const sessionsDir = join(omcRoot, 'state', 'sessions');
+  if (!existsSync(sessionsDir)) return;
+
+  let entries = [];
+  try {
+    entries = readdirSync(sessionsDir);
+  } catch {
+    return;
+  }
+
+  for (const sessionId of entries) {
+    if (!SAFE_SESSION_ID_PATTERN.test(sessionId) || sessionId === currentSessionId) continue;
+
+    const marker = readJsonFile(sessionStartedMarkerPath(omcRoot, sessionId));
+    if (!marker || marker.session_id !== sessionId) continue;
+
+    if (existsSync(join(omcRoot, 'sessions', `${sessionId}.json`))) {
+      removeSessionStartedMarker(omcRoot, sessionId);
+      continue;
+    }
+
+    if (!hasDurableAbandonmentEvidence(marker)) continue;
+
+    cleanupSessionModeState(omcRoot, sessionId);
+    cleanupMissionStateForSession(omcRoot, sessionId);
+    removeSessionStartedMarker(omcRoot, sessionId);
+
+    try {
+      const sessionDir = sessionStateDir(omcRoot, sessionId);
+      if (readdirSync(sessionDir).length === 0) {
+        rmSync(sessionDir, { recursive: false, force: true });
+      }
+    } catch {
+      // Leave non-empty/unreadable directories untouched.
+    }
+  }
+}
+
 function getRuntimeBaseDir() {
   return process.env.PLUGIN_ROOT || process.env.CLAUDE_PLUGIN_ROOT || join(__dirname, '..');
 }
@@ -169,6 +319,21 @@ function semverCompare(a, b) {
 
 const SESSION_START_CONTEXT_BUDGET = 6000;
 const SESSION_START_OMISSION_NOTICE = '[Additional SessionStart context omitted to preserve the 6000-character aggregate budget.]';
+const SESSION_STARTED_MARKER_FILE = 'session-started.json';
+const SAFE_SESSION_ID_PATTERN = /^[a-zA-Z0-9][a-zA-Z0-9_-]{0,255}$/;
+const LINUX_BOOT_ID_PATH = '/proc/sys/kernel/random/boot_id';
+const SESSION_END_MODE_STATE_FILES = [
+  'autopilot-state.json',
+  'autoresearch-state.json',
+  'team-state.json',
+  'ralph-state.json',
+  'ultrawork-state.json',
+  'ultraqa-state.json',
+  'ralplan-state.json',
+  'deep-interview-state.json',
+  'self-improve-state.json',
+  'skill-active-state.json',
+];
 
 function compactBudgetedText(text, maxChars) {
   const notice = '\n...[truncated to preserve SessionStart context budget]';
@@ -456,6 +621,9 @@ async function main() {
     const messages = [];
     const projectMemoryModules = await loadProjectMemoryModules();
 
+    writeSessionStartedMarker(omcRoot, directory, sessionId);
+    reconcileAbandonedSessionStarts(omcRoot, sessionId);
+
     // Check for version drift between components
     const driftInfo = detectVersionDrift();
     if (driftInfo && shouldNotifyDrift(driftInfo)) {

package/skills/AGENTS.md

@@ -3,7 +3,7 @@
 
 # skills
 
-50 skill directories for workflow automation and specialized behaviors.
+30 skill directories for workflow automation and specialized behaviors.
 
 ## Purpose
 
@@ -31,18 +31,7 @@ Skills are reusable workflow templates that can be invoked via `/oh-my-copilot:s
 | `plan/SKILL.md` | omc-plan | Strategic planning with interview workflow |
 | `ralplan/SKILL.md` | ralplan | Iterative planning (Planner+Architect+Critic) with RALPLAN-DR structured deliberation (`--deliberate` for high-risk) |
 | `deep-interview/SKILL.md` | deep-interview | Socratic deep interview with mathematical ambiguity gating (Ouroboros-inspired) |
-| `deep-dive/SKILL.md` | deep-dive | 2-stage pipeline: trace (causal investigation) → deep-interview (requirements crystallization) |
-
-### Analysis Skills
-
-| File | Skill | Purpose |
-|-----------|-------|---------|
-| `critique/SKILL.md` | critique | Pre-push adversarial critique of unpushed commits via independent devils-advocate agent |
-| `deep-review/SKILL.md` | deep-review | Multi-pass code review with security, quality, structural analysis, and validation |
-| `discover/SKILL.md` | discover | Spawn parallel specialist agents to scan the codebase and produce a prioritized improvement backlog |
-| `trace/SKILL.md` | trace | Evidence-driven causal tracing with competing hypotheses |
-| `verify/SKILL.md` | verify | Verify changes work before claiming completion |
-| `debug/SKILL.md` | debug | Diagnose session or repo state using logs, traces, state, and focused reproduction |
+| `ralph-init/SKILL.md` | ralph-init | Initialize PRD for structured ralph |
 
 ### Exploration Skills
 
@@ -50,8 +39,6 @@ Skills are reusable workflow templates that can be invoked via `/oh-my-copilot:s
 |-----------|-------|---------|
 | `deepinit/SKILL.md` | deepinit | Generate hierarchical AGENTS.md |
 | `sciomc/SKILL.md` | sciomc | Parallel scientist orchestration |
-| `external-context/SKILL.md` | external-context | Parallel document-specialist agents for external web searches and documentation lookup |
-| `self-improve/SKILL.md` | self-improve | Autonomous evolutionary code improvement engine with tournament selection |
 
 ### Visual Skills
 
@@ -59,57 +46,23 @@ Skills are reusable workflow templates that can be invoked via `/oh-my-copilot:s
 |-----------|-------|---------|
 | `visual-verdict/SKILL.md` | visual-verdict | Structured visual QA verdict for screenshot/reference comparisons |
 
-### Orchestration Skills
-
-| File | Skill | Purpose |
-|-----------|-------|---------|
-| `cccg/SKILL.md` | cccg | Quadri-model orchestration — Claude, Codex, Gemini provide independent analysis, Copilot synthesizes |
-| `omc-teams/SKILL.md` | omc-teams | CLI-team runtime for claude, codex, or gemini workers in tmux panes |
-| `omc-reference/SKILL.md` | omc-reference | OMC agent catalog, tools, team pipeline routing, commit protocol, and skills registry |
-
-### Notification Skills
-
-| File | Skill | Purpose |
-|-----------|-------|---------|
-| `configure-notifications/SKILL.md` | configure-notifications | Configure notification integrations (Telegram, Discord, Slack, Teams) |
-
-### Azure DevOps Skills
-
-| File | Skill | Purpose |
-|-----------|-------|---------|
-| `omc-ado-setup/SKILL.md` | omc-ado-setup | Configure Azure DevOps integration |
-| `omc-ado-review/SKILL.md` | omc-ado-review | Review Azure DevOps pull requests |
-| `omc-ado-auto-review/SKILL.md` | omc-ado-auto-review | Automated Azure DevOps PR review |
-| `omc-ado-sprint/SKILL.md` | omc-ado-sprint | Azure DevOps sprint management |
-| `omc-ado-triage/SKILL.md` | omc-ado-triage | Azure DevOps work item triage |
-
-### GitHub Skills
-
-| File | Skill | Purpose |
-|-----------|-------|---------|
-| `omc-gh-setup/SKILL.md` | omc-gh-setup | Configure GitHub integration |
-| `omc-gh-triage/SKILL.md` | omc-gh-triage | Issue/PR/CI triage dashboard |
-| `omc-gh-review/SKILL.md` | omc-gh-review | Interactive PR review |
-| `omc-gh-auto-review/SKILL.md` | omc-gh-auto-review | Automated PR code review |
-| `omc-gh-project/SKILL.md` | omc-gh-project | GitHub Projects board management |
-
 ### Utility Skills
 
 | File | Skill | Purpose |
 |-----------|-------|---------|
 | `ai-slop-cleaner/SKILL.md` | ai-slop-cleaner | Regression-safe cleanup workflow for AI-generated code slop |
-| `learner/SKILL.md` | learner | Extract reusable skill from session |
-| `ask/SKILL.md` | ask | Ask Copilot, Codex, or Gemini via `omcp ask` and capture an artifact |
+| `skillify/SKILL.md` | skillify | Extract reusable skill from session |
+| `learner/SKILL.md` | learner | Deprecated compatibility alias/internal implementation history for skillify |
+| `ask/SKILL.md` | ask | Ask Claude, Codex, or Gemini via `omc ask` and capture an artifact |
+| `note/SKILL.md` | note | Save notes for compaction resilience |
 | `cancel/SKILL.md` | cancel | Cancel any active OMC mode |
-| `hud/SKILL.md` | hud | Configure HUD display options (layout, presets, display elements) |
+| `hud/SKILL.md` | hud | Configure HUD display |
 | `omc-doctor/SKILL.md` | omc-doctor | Diagnose installation issues |
-| `remember/SKILL.md` | remember | Review reusable project knowledge and decide what belongs in project memory, notepad, or docs |
 | `setup/SKILL.md` | setup | Unified setup entrypoint for install, diagnostics, and MCP configuration |
 | `omc-setup/SKILL.md` | omc-setup | One-time setup wizard |
+| `omc-help/SKILL.md` | omc-help | Usage guide |
 | `mcp-setup/SKILL.md` | mcp-setup | Configure MCP servers |
 | `skill/SKILL.md` | skill | Manage local skills |
-| `skillify/SKILL.md` | skillify | Turn a repeatable workflow into a reusable OMC skill draft |
-| `wiki/SKILL.md` | wiki | Persistent markdown knowledge base that compounds across sessions |
 
 ### Domain Skills
 
@@ -117,22 +70,7 @@ Skills are reusable workflow templates that can be invoked via `/oh-my-copilot:s
 |-----------|-------|---------|
 | `project-session-manager/SKILL.md` | project-session-manager (+ `psm` alias) | Isolated dev environments |
 | `writer-memory/SKILL.md` | writer-memory | Agentic memory for writers |
-| `release/SKILL.md` | release | Automated release workflow |
-| `ralph-experiment/SKILL.md` | ralph-experiment | Hypothesis-driven experiment loop with structured notebook and git checkpoints |
-
-## Permission Model
-
-All plugin MCP tools use a **three-tier auto-approval system** so agents can operate without `/yolo`:
-
-- **Tier 1** (read-only): LSP nav, code search, state/memory reads — always auto-approved via `readOnlyHint` annotation
-- **Tier 2** (write): State/memory writes, AST replace, LSP rename — auto-approved via `permissions.allow` in `settings.local.json`
-- **Tier 3** (destructive): `shared_memory_delete`, `shared_memory_cleanup`, `kill_job` — always prompt user
-
-Safe bash patterns (git, npm, dotnet, gh, az, grep, find, ls) are auto-approved by the `permissionRequest` hook. Shell metacharacters are always rejected.
-
-Deny escalation: 3 consecutive or 20 total denials stops the session. All decisions are audit-logged to `.omcp/logs/permissions.log`.
-
-See [permission architecture](../docs/architecture/permissions.md) for details.
+| `release/SKILL.md` | release | Generic release assistant — analyzes repo CI/rules, caches in `.omcp/RELEASE_RULE.md`, guides the release |
 
 ## For AI Agents
 
@@ -149,6 +87,10 @@ triggers:
   - "keyword2"
 agent: executor  # Optional: which agent to use
 model: sonnet    # Optional: model override
+pipeline: [skill-name, follow-up-skill]  # Optional: standardized multi-skill flow
+next-skill: follow-up-skill              # Optional: explicit handoff target
+next-skill-args: --direct                # Optional: arguments for the next skill
+handoff: .omcp/plans/example.md           # Optional: artifact/context handed to next skill
 ---
 
 # Skill Name
@@ -187,7 +129,7 @@ Any configurable options.
 2. Define purpose, workflow, and usage
 3. Add to skill registry (auto-detected from frontmatter)
 4. Optionally add activation triggers
-5. Create corresponding `commands/new-skill.md` file (mirror)
+5. Create corresponding plugin-scoped skill/slash surface via `skills/new-skill/SKILL.md` (and generated artifacts when the build requires them)
 6. Update `docs/REFERENCE.md` (Skills section, count)
 7. If execution mode skill, also create `src/hooks/new-skill/` hook
 
@@ -202,6 +144,8 @@ Any configurable options.
 4. Invoke `qa-tester` for verification
 ```
 
+If `pipeline` / `next-skill` metadata is present, OMC appends a standardized **Skill Pipeline** handoff block to the rendered skill prompt so downstream steps are explicit.
+
 **Conditional behavior:**
 ```markdown
 ## Workflow
@@ -232,18 +176,12 @@ None - pure markdown files.
 
 | Category | Skills | Trigger Keywords |
 |----------|--------|------------------|
-| Execution | autopilot, ultrawork, ralph, team, ultraqa, ralph-experiment, self-improve | "autopilot", "ulw", "ralph", "self-improve" |
-| Planning | omc-plan, ralplan, deep-interview, deep-dive | "ralplan", "deep-interview", "ouroboros" |
-| Analysis | critique, deep-review, discover, debug, trace, verify | "critique", "code review", "security review", "trace", "verify", "debug" |
-| Exploration | deepinit, sciomc, external-context | "deepinit", "sciomc", "external-context" |
-| Orchestration | cccg, omc-teams, omc-reference | "cccg", "ask codex", "ask gemini", "ask claude" |
-| Visual | visual-verdict | screenshot QA |
-| Cleanup | ai-slop-cleaner | "deslop", "anti-slop" |
-| Azure DevOps | omc-ado-setup, omc-ado-review, omc-ado-auto-review, omc-ado-sprint, omc-ado-triage | "ado setup", "ado triage" |
-| GitHub | omc-gh-setup, omc-gh-triage, omc-gh-review, omc-gh-auto-review, omc-gh-project | "gh setup", "gh triage", "gh review" |
-| Notifications | configure-notifications | "configure discord", "setup telegram" |
-| Utility | learner, ask, cancel, setup, omc-doctor, omc-setup, mcp-setup, skill, skillify, wiki, hud, remember | "cancelomc", "stopomc", "wiki", "skillify" |
-| Domain | project-session-manager, writer-memory, release | psm context |
+| Execution | autopilot, ultrawork, ralph, team, ultraqa | "autopilot", "ulw", "ralph", "team" |
+| Cleanup | ai-slop-cleaner | "deslop", "anti-slop", cleanup/refactor + slop smells |
+| Planning | omc-plan, ralplan, deep-interview, ralph-init | "plan this", "interview me", "ouroboros" |
+| Exploration | deepinit, sciomc, external-context | "deepinit", "research" |
+| Utility | skillify, learner (deprecated alias), note, cancel, hud, setup, omc-doctor, omc-setup, omc-help, mcp-setup | "stop", "cancel" |
+| Domain | psm, writer-memory, release | psm context |
 
 ## Auto-Activation
 
@@ -255,13 +193,6 @@ Some skills activate automatically based on context:
 | ultrawork | "ulw", "ultrawork" |
 | ralph | "ralph", "don't stop until" |
 | deep-interview | "deep interview", "interview me", "ouroboros", "don't assume" |
-| critique | "critique", "review before push", "critique my changes" |
-| self-improve | "self-improve" |
-| ai-slop-cleaner | "deslop", "anti-slop", "clean slop" |
-| discover | "discover" |
-| trace | "trace" |
-| verify | "verify" |
-| wiki | "wiki" |
 | cancel | "stop", "cancel", "abort" |
 
 <!-- MANUAL: