company-skill 4.5.0 → 4.5.1

package/README.md

@@ -27,17 +27,17 @@ Optionally define your team first in `COMPANY.md` (skip it and a minimal company
 ```mermaid
 graph LR
     G[GOAL] --> T[THINK]
-    T -->|contract shape gate| E[EXECUTE]
+    T -->|contract shape gate| E[EXECUTE in dependency waves]
     E -->|findings shape gate| V[VERIFY]
     V -->|reviewer re-derives + critic attacks| D{Done?}
     D -->|NO: feedback| C[COMPRESS]
     C --> T
-    D -->|YES| S[STATUS.md]
-    D -.->|stop attempt while failing| B[stop guard blocks]
+    D -->|YES| S[STATUS.md + playbook]
+    D -.->|stop blocked with the goal and failing notes| B[stop guard]
     B -.-> T
 ```
 
-It runs any model the way frontier models like Claude Fable 5 run themselves: delegation contracts, verify layers, and failing-by-default criteria ship as structural artifacts, so the discipline holds whichever model fills each role. The orchestrator reads the goal and activates only the relevant employees. Leads decompose the goal into delegation contracts, workers execute them in parallel waves, and two reviewers gate every cycle: the Internal Reviewer re-runs the evidence and the Devil's Advocate attacks it. There is no iteration limit. The harness carries the quality, so none of it depends on the model remembering to be careful.
+It runs any model with the operating rigor the frontier pair Claude Fable 5 and Mythos demonstrate: delegation contracts, two-pass evidence verification, and failing-by-default criteria ship as structural artifacts, so the discipline holds whichever model fills each role. The orchestrator reads the goal and activates only the relevant employees. Leads decompose the goal into delegation contracts, workers execute them in parallel waves, and two reviewers gate every cycle: the Internal Reviewer re-runs the evidence and the Devil's Advocate attacks it. There is no iteration limit. The harness carries the quality, so none of it depends on the model remembering to be careful.
 
 ## Delegation contracts
 

package/agents/company-worker.md

@@ -33,6 +33,8 @@ Rate each finding's importance 1-5 (the digest keeps 4-5 in full).
 
 Report SHORT. Result first, then the evidence (FINDING + SOURCE: the command and its output, the file, the PR/SHA/CI link). No narration of your steps, no restating the task. Concise never means unsourced: cut the prose around a claim, never the source that proves it.
 
+Narrate intent before consequential tool calls: one short line on what you are about to do and why. A silent agent is far harder for the verify layers to audit, and the audit trail is the product.
+
 Anything a human reads outside the run (a PR body, a comment, an email, a post) gets a /humanizer pass before you publish it: short, professional, human-sounding. Evidence lines stay verbatim. If the skill is missing, self-edit to the same bar and note SKILL-MISSING.
 
 End every findings append with one machine-greppable line: `STATUS: complete` when DONE-WHEN is met and verified, `STATUS: blocked` with the blocker named above it, or `STATUS: incomplete` with what remains. The orchestrator greps this line instead of parsing your prose.

package/bin/install.js

@@ -16,22 +16,18 @@ function copyFile(src, dest) {
   fs.copyFileSync(src, dest);
 }
 
-// Install skill
 copyFile(path.join(srcDir, 'skill', 'SKILL.md'), path.join(skillDir, 'SKILL.md'));
 
-// Install commands
 for (const cmd of ['run', 'status', 'resume']) {
   const src = path.join(srcDir, 'commands', `${cmd}.md`);
   if (fs.existsSync(src)) copyFile(src, path.join(commandsDir, `${cmd}.md`));
 }
 
-// Install agents
 for (const agent of ['lead', 'worker', 'reviewer', 'critic', 'digest']) {
   const src = path.join(srcDir, 'agents', `company-${agent}.md`);
   if (fs.existsSync(src)) copyFile(src, path.join(agentsDir, `company-${agent}.md`));
 }
 
-// Install all hooks
 const hookFiles = {
   'stop-guard.js': 'company-stop-guard.js',
   'precompact.js': 'company-precompact.js',
@@ -52,19 +48,16 @@ try {
 
   if (!settings.hooks) settings.hooks = {};
 
-  // Stop hook
   if (!settings.hooks.Stop) settings.hooks.Stop = [];
   if (!settings.hooks.Stop.some(h => h.hooks?.some(hh => hh.command?.includes('company-stop-guard')))) {
     settings.hooks.Stop.push({ hooks: [{ type: 'command', command: `node "${path.join(hooksDir, 'company-stop-guard.js')}"`, timeout: 10 }] });
   }
 
-  // PreCompact hook
   if (!settings.hooks.PreCompact) settings.hooks.PreCompact = [];
   if (!settings.hooks.PreCompact.some(h => h.hooks?.some(hh => hh.command?.includes('company-precompact')))) {
     settings.hooks.PreCompact.push({ hooks: [{ type: 'command', command: `node "${path.join(hooksDir, 'company-precompact.js')}"`, timeout: 10 }] });
   }
 
-  // SessionStart hook (compact restore)
   if (!settings.hooks.SessionStart) settings.hooks.SessionStart = [];
   if (!settings.hooks.SessionStart.some(h => h.hooks?.some(hh => hh.command?.includes('company-session-restore')))) {
     settings.hooks.SessionStart.push({ matcher: 'compact', hooks: [{ type: 'command', command: `node "${path.join(hooksDir, 'company-session-restore.js')}"`, timeout: 10 }] });
@@ -79,7 +72,6 @@ try {
   console.log('Could not register hooks. Add manually to settings.json.');
 }
 
-// Create COMPANY.md template
 const companyMd = path.join(process.cwd(), 'COMPANY.md');
 const template = path.join(srcDir, 'COMPANY.md.template');
 if (!fs.existsSync(companyMd) && fs.existsSync(template)) {
@@ -87,7 +79,6 @@ if (!fs.existsSync(companyMd) && fs.existsSync(template)) {
   console.log('Created COMPANY.md template.');
 }
 
-// Gitignore
 try {
   const gi = path.join(process.cwd(), '.gitignore');
   const content = fs.existsSync(gi) ? fs.readFileSync(gi, 'utf8') : '';

package/hooks/precompact.js

@@ -1,7 +1,8 @@
 #!/usr/bin/env node
 
-// Save company state AND reasoning before context compaction.
-// Saves both numbers (criteria, cycle) and intent (what we're trying and why).
+// Saves company state before context compaction so session-restore can rebuild
+// the model's context. Snapshots goal, cycle, briefing, review, criteria, roster,
+// and the TAIL of the playbook (new entries append at the bottom).
 
 const fs = require('fs');
 const path = require('path');
@@ -9,9 +10,9 @@ const path = require('path');
 const companyDir = process.env.COMPANY_DIR || path.join(process.cwd(), '.company');
 if (!fs.existsSync(companyDir)) process.exit(0);
 
-// Only sessions that own the run are acted on. A foreign session that merely
-// shares the directory must not be redirected or have state written on its
-// behalf. Missing or empty OWNER is legacy state and keeps the old behavior.
+// Only sessions listed in OWNER are acted on. A foreign session that shares the
+// directory must not have state written on its behalf. Missing or empty OWNER
+// keeps the old behavior (act on all sessions).
 try {
   const hookInput = JSON.parse(fs.readFileSync(0, 'utf8'));
   if (hookInput && typeof hookInput.session_id === 'string') {
@@ -23,7 +24,6 @@ try {
 
 const lines = ['# Company Checkpoint (auto-saved before compaction)', ''];
 
-// Goal
 const goalPath = path.join(companyDir, 'GOAL.md');
 if (fs.existsSync(goalPath)) {
   lines.push('## Goal');
@@ -31,7 +31,6 @@ if (fs.existsSync(goalPath)) {
   lines.push('');
 }
 
-// Cycle number
 const cyclesDir = path.join(companyDir, 'cycles');
 if (fs.existsSync(cyclesDir)) {
   const files = fs.readdirSync(cyclesDir).filter(f => f.startsWith('cycle-'));
@@ -40,7 +39,6 @@ if (fs.existsSync(cyclesDir)) {
   lines.push('## Cycle: ' + cycle);
   lines.push('');
 
-  // Latest briefing (captures current reasoning/intent)
   const briefing = path.join(cyclesDir, `cycle-${cycle}-briefing.md`);
   if (fs.existsSync(briefing)) {
     lines.push('## Current Reasoning');
@@ -48,7 +46,6 @@ if (fs.existsSync(cyclesDir)) {
     lines.push('');
   }
 
-  // Latest review (captures what's working/failing)
   const review = path.join(cyclesDir, `cycle-${cycle}-review.md`);
   if (fs.existsSync(review)) {
     lines.push('## Latest Review');
@@ -57,7 +54,6 @@ if (fs.existsSync(cyclesDir)) {
   }
 }
 
-// Criteria status
 const criteriaPath = path.join(companyDir, 'criteria.json');
 if (fs.existsSync(criteriaPath)) {
   try {
@@ -71,7 +67,6 @@ if (fs.existsSync(criteriaPath)) {
   } catch (e) {}
 }
 
-// Active roster
 const rosterPath = path.join(companyDir, 'active-roster.md');
 if (fs.existsSync(rosterPath)) {
   lines.push('## Active Roster');
@@ -79,8 +74,7 @@ if (fs.existsSync(rosterPath)) {
   lines.push('');
 }
 
-// Playbook (accumulated lessons). New session entries are appended at the
-// bottom, so snapshot the TAIL, not the head.
+// New playbook entries append at the bottom, so snapshot the tail, not the head.
 const playbookPath = path.join(companyDir, 'playbook.md');
 if (fs.existsSync(playbookPath)) {
   const playbook = fs.readFileSync(playbookPath, 'utf8');

package/hooks/session-restore.js

@@ -1,14 +1,10 @@
 #!/usr/bin/env node
 
-// Restore company state after compaction and drive the /company restart handoff.
-// PreCompact cannot make the model emit a prompt (shell-only, no model turn before
-// compaction), so the reliable trigger is here: right after compaction the model is
-// instructed to run /company restart (its mandatory verify and debate procedure)
-// and emit the handoff.
-//
-// SessionStart context must go through hookSpecificOutput.additionalContext.
-// That field reaches the model. systemMessage is a user-facing display only and
-// never enters the model's context.
+// Restores company context after compaction and instructs the model to run the
+// /company restart procedure before continuing.
+// Context must go through hookSpecificOutput.additionalContext - that field
+// reaches the model. systemMessage is user-facing display only and never enters
+// the model's context.
 
 const fs = require('fs');
 const path = require('path');
@@ -16,9 +12,8 @@ const path = require('path');
 const companyDir = process.env.COMPANY_DIR || path.join(process.cwd(), '.company');
 if (!fs.existsSync(companyDir)) process.exit(0);
 
-// Only sessions that own the run are acted on. A foreign session that merely
-// shares the directory must not be redirected or have state written on its
-// behalf. Missing or empty OWNER is legacy state and keeps the old behavior.
+// Only sessions listed in OWNER are acted on. A foreign session that shares the
+// directory must not be redirected. Missing or empty OWNER keeps the old behavior.
 try {
   const hookInput = JSON.parse(fs.readFileSync(0, 'utf8'));
   if (hookInput && typeof hookInput.session_id === 'string') {
@@ -34,7 +29,6 @@ if (fs.existsSync(checkpointMd)) {
   state = fs.readFileSync(checkpointMd, 'utf8').substring(0, 2000);
 }
 
-// The post-compaction directive: run the restart procedure, do not just "continue".
 const directive =
   '[COMPANY] Context was compacted, so prior turn-by-turn state is gone. Before doing ' +
   'anything else, run the /company restart procedure from the skill: refresh ' +

package/install.sh

@@ -1,6 +1,6 @@
 #!/bin/bash
-# Installs the /company skill, commands, agents, AND hooks. Idempotent:
-# re-running overwrites the copied files and never duplicates hook entries.
+# Installs the /company skill, commands, agents, and hooks.
+# Idempotent: re-running overwrites copied files and never duplicates hook entries.
 set -e
 
 REPO="https://raw.githubusercontent.com/jagmarques/company-skill/main"
@@ -33,7 +33,7 @@ for agent in lead worker reviewer critic digest; do
   fetch "$REPO/agents/company-$agent.md" "$HOME/.claude/agents/company-$agent.md" || echo "Warning: failed to download agent company-$agent"
 done
 
-# Hooks (the stop gate and the compaction machinery live here)
+# Hooks
 mkdir -p "$HOME/.claude/hooks"
 for pair in "stop-guard company-stop-guard" "precompact company-precompact" "session-restore company-session-restore"; do
   src="${pair%% *}"
@@ -41,9 +41,8 @@ for pair in "stop-guard company-stop-guard" "precompact company-precompact" "ses
   fetch "$REPO/hooks/$src.js" "$HOME/.claude/hooks/$dest.js" || echo "Warning: failed to download hook $src"
 done
 
-# Register hooks in ~/.claude/settings.json. JSON editing from shell is not
-# safe without a JSON tool, so use node when available and print exact manual
-# steps otherwise. The merge is idempotent: existing entries are kept.
+# JSON editing from shell is unsafe without a JSON tool. Use node when available
+# and print exact manual steps otherwise. The merge is idempotent.
 if command -v node >/dev/null 2>&1; then
   node <<'EOF'
 const fs = require('fs');

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "company-skill",
-  "version": "4.5.0",
+  "version": "4.5.1",
   "description": "Goal-driven multi-employee company for Claude Code. Give it a goal, it runs until done.",
   "bin": {
     "company-skill": "./bin/install.js"
   },
-  "keywords": ["claude-code", "skill", "multi-agent", "company", "orchestration", "model-agnostic", "autonomous-agents", "stop-hook", "agent-orchestration", "ai-agents", "fable", "claude-fable"],
+  "keywords": ["claude-code", "skill", "multi-agent", "company", "orchestration", "model-agnostic", "autonomous-agents", "stop-hook", "agent-orchestration", "ai-agents", "fable", "claude-fable", "mythos", "claude-mythos"],
   "author": "jagmarques",
   "license": "MIT",
   "repository": {

package/scripts/check-contracts.js

@@ -1,8 +1,7 @@
 #!/usr/bin/env node
-// Mechanical gate for delegation contracts: every TASK block must carry all
-// seven fields and a non-empty VERIFY-WITH. Run before EXECUTE:
-//   node scripts/check-contracts.js .company/cycles/cycle-N-tasks.md
-// Exit 1 lists each defective contract. No fields, no task.
+// Gate for delegation contracts: every TASK block must carry all seven fields
+// and a non-empty VERIFY-WITH. Exit 1 lists each defective contract.
+// Run: node scripts/check-contracts.js .company/cycles/cycle-N-tasks.md
 const fs = require('fs');
 const file = process.argv[2];
 if (!file || !fs.existsSync(file)) { console.error('usage: check-contracts.js <tasks-file>'); process.exit(2); }
@@ -10,8 +9,8 @@ const text = fs.readFileSync(file, 'utf8');
 const blocks = text.split(/\n(?=TASK:)/).filter(b => b.trim().startsWith('TASK:'));
 if (blocks.length === 0) { console.error('no TASK blocks found'); process.exit(1); }
 const FIELDS = ['TASK:', 'EMPLOYEE:', 'SKILL:', 'INPUTS:', 'OUTPUT:', 'DONE-WHEN:', 'VERIFY-WITH:', 'OUT-OF-SCOPE:'];
-// DEPENDS-ON is optional. When present it must name existing task numbers
-// and the dependency graph must be acyclic.
+// DEPENDS-ON is optional. When present it must name existing task numbers and the
+// dependency graph must be acyclic.
 function checkDeps(blocks) {
   const errs = [];
   const deps = blocks.map((b, i) => {

package/scripts/check-findings.js

@@ -1,8 +1,7 @@
 #!/usr/bin/env node
-// Mechanical gate for findings files: every FINDING line must be followed by
-// a SOURCE line (or NOVEL marker) before the next FINDING. Run at VERIFY:
-//   node scripts/check-findings.js .company/<dept>/<employee>.md [...]
-// Exit 1 names each unsourced finding. A claim without a source is unverifiable.
+// Gate for findings files: every FINDING line must be followed by a SOURCE
+// line (or NOVEL marker) before the next FINDING. A claim without a source
+// is unverifiable. Run: node scripts/check-findings.js .company/<dept>/<employee>.md [...]
 const fs = require('fs');
 const files = process.argv.slice(2);
 if (!files.length) { console.error('usage: check-findings.js <findings-file...>'); process.exit(2); }

package/tests/stop-guard.test.js

@@ -1,11 +1,9 @@
 #!/usr/bin/env node
 
-// Decision-logic matrix for hooks/stop-guard.js. Each case builds a fixture
-// state dir, runs the hook with COMPANY_DIR pointed at it, and asserts the
-// allow/block decision from the hook's stdout contract: a block prints a
-// JSON object with decision "block", an allow prints nothing and exits 0.
-// The CI floor (scripts/check.sh) runs this file, so a future edit cannot
-// regress the fail-closed behavior with CI still green.
+// Decision-logic matrix for hooks/stop-guard.js. Builds a fixture dir per case,
+// runs the hook with COMPANY_DIR pointed at it, and asserts allow/block from
+// stdout. A block emits JSON with decision "block"; allow exits 0 with no output.
+// Included in scripts/check.sh so a future edit cannot silently regress fail-closed.
 
 const { execFileSync } = require('child_process');
 const fs = require('fs');
@@ -65,13 +63,13 @@ function writeCriteria(dir, value) {
     typeof value === 'string' ? value : JSON.stringify(value));
 }
 
-// 1. No company state at all: the hook must stay silent (allow).
+// 1. No company state: hook must allow silently.
 {
   const d = freshDir();
   check('no state allows', d, 'allow');
 }
 
-// 2. CANCEL file: allows the stop AND is consumed so it cannot leak into a later run.
+// 2. CANCEL file allows and must be consumed so it cannot leak into a later run.
 {
   const d = freshDir();
   fs.writeFileSync(path.join(d, 'GOAL.md'), 'goal');
@@ -114,14 +112,14 @@ function writeCriteria(dir, value) {
   check('failing criterion blocks', d, 'block', 'ship it');
 }
 
-// 7. passes true with null evidence still blocks: evidence is the contract.
+// 7. passes:true without evidence still blocks; evidence is the contract.
 {
   const d = freshDir();
   writeCriteria(d, { criteria: [{ id: 1, description: 'ev gap', passes: true, evidence: null }] });
   check('passes without evidence blocks', d, 'block', 'ev gap');
 }
 
-// 8. All passing with evidence allows the stop.
+// 8. All criteria passing with evidence allows the stop.
 {
   const d = freshDir();
   writeCriteria(d, { criteria: [
@@ -154,7 +152,7 @@ function writeCriteria(dir, value) {
   check('stale still blocks with age note', d, 'block', 'untouched for');
 }
 
-// 12. Session scoping: a foreign session passes when OWNER names another session.
+// 12. A foreign session passes when OWNER names a different session.
 {
   const d = freshDir();
   writeCriteria(d, { criteria: [{ id: 1, description: 'a', passes: false, evidence: null }] });
@@ -201,7 +199,7 @@ function writeCriteria(dir, value) {
     { input: JSON.stringify({ session_id: 'owner-eee' }) });
 }
 
-// 17. Deleting a locked criterion blocks even when everything left passes.
+// 17. Deleting a locked criterion blocks even when everything remaining passes.
 {
   const d = freshDir();
   writeCriteria(d, { criteria: [
@@ -213,7 +211,7 @@ function writeCriteria(dir, value) {
   check('deleting a locked criterion blocks', d, 'block', 'locked criterion');
 }
 
-// 18. Adding a criterion extends the lock and a fully passing set allows.
+// 18. Adding a criterion extends the lock; a fully passing set allows.
 {
   const d = freshDir();
   writeCriteria(d, { criteria: [
@@ -233,7 +231,7 @@ function writeCriteria(dir, value) {
   }
 }
 
-// 19. No block reason ever hands the model the cancel command.
+// 19. The block reason must not reveal the cancel command to the model.
 {
   const d = freshDir();
   writeCriteria(d, { criteria: [{ id: 1, description: 'a', passes: false, evidence: null }] });