@erclx/canon 5.26.0 → 5.28.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +2 -2
- package/claude/.claude-plugin/plugin.json +1 -1
- package/claude/skills/api-design/SKILL.md +2 -0
- package/claude/skills/auto-ship/SKILL.md +3 -11
- package/claude/skills/auto-ship/references/review-findings.md +20 -0
- package/claude/skills/backlog-triage/SKILL.md +2 -0
- package/claude/skills/bash-cli-script/SKILL.md +2 -0
- package/claude/skills/build-in-slices/SKILL.md +2 -0
- package/claude/skills/canon-cli/SKILL.md +2 -0
- package/claude/skills/canon-feedback/SKILL.md +2 -0
- package/claude/skills/canon-feedback-triage/SKILL.md +2 -0
- package/claude/skills/canon-operator/SKILL.md +2 -0
- package/claude/skills/canon-rollout/SKILL.md +3 -11
- package/claude/skills/canon-rollout/references/failure-recovery.md +20 -0
- package/claude/skills/canvas/SKILL.md +13 -0
- package/claude/skills/ci-workflow/SKILL.md +2 -0
- package/claude/skills/code-craft/SKILL.md +2 -0
- package/claude/skills/codebase-layout/SKILL.md +2 -0
- package/claude/skills/context-fold/SKILL.md +3 -17
- package/claude/skills/context-fold/references/completion-report.md +26 -0
- package/claude/skills/create-rule/SKILL.md +2 -0
- package/claude/skills/create-skill/SKILL.md +2 -0
- package/claude/skills/create-standard/SKILL.md +2 -0
- package/claude/skills/decision-escalate/SKILL.md +2 -0
- package/claude/skills/deploy-app/SKILL.md +2 -0
- package/claude/skills/deprecation-migration/SKILL.md +2 -0
- package/claude/skills/design-extract/SKILL.md +2 -0
- package/claude/skills/design-taste/SKILL.md +2 -0
- package/claude/skills/docs-sync/SKILL.md +2 -0
- package/claude/skills/document-health/SKILL.md +2 -0
- package/claude/skills/draft-and-pick/SKILL.md +2 -0
- package/claude/skills/draft-doc/SKILL.md +2 -0
- package/claude/skills/draft-doc/references/wiki.md +1 -1
- package/claude/skills/draft-figure/SKILL.md +2 -0
- package/claude/skills/draft-identity/SKILL.md +2 -0
- package/claude/skills/draft-ready/SKILL.md +5 -1
- package/claude/skills/draft-screencast/SKILL.md +2 -0
- package/claude/skills/draft-slides/SKILL.md +2 -0
- package/claude/skills/git-branch/SKILL.md +2 -0
- package/claude/skills/git-commit/SKILL.md +2 -0
- package/claude/skills/git-followup/SKILL.md +2 -0
- package/claude/skills/git-issue/SKILL.md +2 -0
- package/claude/skills/git-pr/SKILL.md +2 -0
- package/claude/skills/git-pr/references/evidence.md +7 -7
- package/claude/skills/git-ship/SKILL.md +2 -0
- package/claude/skills/git-split/SKILL.md +2 -0
- package/claude/skills/git-stage/SKILL.md +2 -0
- package/claude/skills/git-worktree/SKILL.md +2 -0
- package/claude/skills/index-lookup/SKILL.md +2 -0
- package/claude/skills/markdown-propose/SKILL.md +2 -0
- package/claude/skills/memory-capture/SKILL.md +2 -0
- package/claude/skills/memory-review/SKILL.md +2 -0
- package/claude/skills/plan-feature/SKILL.md +2 -0
- package/claude/skills/plan-groundwork/SKILL.md +2 -0
- package/claude/skills/plan-intake/SKILL.md +2 -0
- package/claude/skills/plan-intake-answer/SKILL.md +2 -0
- package/claude/skills/project-commands/SKILL.md +2 -0
- package/claude/skills/read-frames/SKILL.md +2 -0
- package/claude/skills/record-screencast/SKILL.md +2 -0
- package/claude/skills/repo-metadata/SKILL.md +2 -0
- package/claude/skills/restate-plainly/SKILL.md +2 -0
- package/claude/skills/review-address/REQUIREMENT.md +4 -2
- package/claude/skills/review-address/SKILL.md +19 -14
- package/claude/skills/review-address/references/post-review-findings.md +16 -0
- package/claude/skills/review-address/references/rebase-conflicts.md +4 -2
- package/claude/skills/review-branch/SKILL.md +2 -0
- package/claude/skills/review-craft/SKILL.md +2 -0
- package/claude/skills/review-craft/references/ui.md +6 -6
- package/claude/skills/review-pr/SKILL.md +4 -8
- package/claude/skills/review-pr/references/body-variants.md +52 -0
- package/claude/skills/review-pr/references/key-changes.md +16 -0
- package/claude/skills/review-pr/references/post.md +15 -43
- package/claude/skills/review-ui/REQUIREMENT.md +2 -2
- package/claude/skills/review-ui/SKILL.md +16 -12
- package/claude/skills/review-ui/references/driving.md +8 -8
- package/claude/skills/review-ui/references/post.md +13 -3
- package/claude/skills/review-ui/references/tick.md +22 -0
- package/claude/skills/role-orchestrator/REQUIREMENT.md +15 -15
- package/claude/skills/role-orchestrator/SKILL.md +31 -42
- package/claude/skills/role-orchestrator/references/orchestrator-dispatch.md +18 -16
- package/claude/skills/role-orchestrator/references/orchestrator-handback.md +13 -3
- package/claude/skills/role-orchestrator/references/orchestrator-inbound.md +23 -0
- package/claude/skills/role-orchestrator/references/orchestrator-launch.md +6 -34
- package/claude/skills/role-orchestrator/references/orchestrator-output.md +4 -4
- package/claude/skills/role-orchestrator/references/orchestrator-parked.md +5 -5
- package/claude/skills/role-orchestrator/references/orchestrator-poll.md +12 -23
- package/claude/skills/role-orchestrator/references/orchestrator-refill.md +6 -6
- package/claude/skills/role-orchestrator/references/orchestrator-review-dispatch.md +1 -1
- package/claude/skills/role-orchestrator/references/orchestrator-reviewer-launch.md +38 -0
- package/claude/skills/role-orchestrator/references/orchestrator-sweep.md +2 -2
- package/claude/skills/role-orchestrator/references/orchestrator-watch.md +26 -0
- package/claude/skills/role-orchestrator/scripts/poll.ts +304 -10
- package/claude/skills/role-planner/REQUIREMENT.md +1 -1
- package/claude/skills/role-planner/SKILL.md +4 -2
- package/claude/skills/role-reviewer/REQUIREMENT.md +1 -1
- package/claude/skills/role-reviewer/SKILL.md +3 -1
- package/claude/skills/role-worker/SKILL.md +5 -2
- package/claude/skills/search-craft/SKILL.md +2 -0
- package/claude/skills/seed-sync/SKILL.md +2 -0
- package/claude/skills/session-compact/SKILL.md +2 -0
- package/claude/skills/session-map/SKILL.md +2 -0
- package/claude/skills/session-relay/SKILL.md +2 -0
- package/claude/skills/session-resume/SKILL.md +2 -0
- package/claude/skills/session-worktree/SKILL.md +3 -11
- package/claude/skills/session-worktree/references/install-command.md +20 -0
- package/claude/skills/sketch-design/SKILL.md +2 -0
- package/claude/skills/systematic-debugging/SKILL.md +2 -0
- package/claude/skills/target-check/SKILL.md +2 -0
- package/claude/skills/target-setup/SKILL.md +2 -0
- package/claude/skills/task-board/SKILL.md +2 -0
- package/claude/skills/teach-workspace/SKILL.md +5 -15
- package/claude/skills/teach-workspace/references/lesson-build.md +75 -0
- package/claude/skills/teach-workspace/references/lesson-craft.md +13 -69
- package/claude/skills/teach-workspace/references/revisit.md +22 -0
- package/claude/skills/test-craft/SKILL.md +2 -0
- package/claude/skills/test-first/SKILL.md +2 -0
- package/claude/skills/ui-checklist/SKILL.md +3 -1
- package/claude/skills/ux-audit/SKILL.md +2 -0
- package/claude/skills/ux-measure/SKILL.md +2 -0
- package/claude/skills/ux-walkthrough/SKILL.md +2 -0
- package/claude/skills/video-craft/SKILL.md +2 -0
- package/claude/skills/write-human/SKILL.md +2 -0
- package/claude/skills/youtube-transcripts/SKILL.md +2 -0
- package/docs/agents/canvas.md +24 -8
- package/docs/agents/capture.md +5 -5
- package/docs/agents/commands.md +2 -2
- package/docs/agents/gate.md +3 -1
- package/docs/agents/index.md +5 -4
- package/docs/agents/install-and-sync.md +7 -0
- package/docs/agents/key-changes.md +1 -1
- package/docs/agents/pr-evidence.md +47 -23
- package/docs/agents/pr-local.md +19 -13
- package/docs/agents/pr-preview.md +44 -21
- package/docs/agents/pr-tick.md +72 -0
- package/docs/agents/sandbox.md +30 -6
- package/docs/agents/scripting.md +8 -1
- package/docs/agents/skills-audit.md +2 -1
- package/docs/agents/tasks-plans.md +1 -1
- package/docs/agents/tasks-validate.md +18 -18
- package/docs/agents/tasks.md +1 -1
- package/docs/workflow/operating-model.md +15 -21
- package/docs/workflow/parallel-features.md +24 -3
- package/docs/workflow/skill-map.md +3 -1
- package/docs/workflow/zshrc-aliases.md +1 -1
- package/package.json +1 -1
- package/scripts/core/regen-hero.ts +1 -1
- package/scripts/lib/sandbox-path.sh +6 -162
- package/src/audits/baseline.ts +1 -1
- package/src/audits/run.ts +2 -2
- package/src/canvas/address.ts +12 -3
- package/src/canvas/capture.ts +1 -1
- package/src/canvas/client/inspector/align-grid.tsx +162 -0
- package/src/canvas/client/inspector/color-field.tsx +201 -171
- package/src/canvas/client/inspector/color-inputs.tsx +139 -0
- package/src/canvas/client/inspector/color-picker.tsx +668 -0
- package/src/canvas/client/inspector/color-space.ts +240 -0
- package/src/canvas/client/inspector/field.tsx +66 -3
- package/src/canvas/client/inspector/fields.css +473 -6
- package/src/canvas/client/inspector/size-field.tsx +69 -0
- package/src/canvas/client/inspector/token-select.tsx +109 -0
- package/src/canvas/client/inspector/values.ts +117 -3
- package/src/canvas/client/inspector.tsx +311 -15
- package/src/canvas/client/keys.ts +54 -0
- package/src/canvas/client/selection.tsx +199 -0
- package/src/canvas/client/shell.css +210 -12
- package/src/canvas/client/state.ts +165 -14
- package/src/canvas/client/surface.tsx +341 -39
- package/src/canvas/client/tools.tsx +64 -0
- package/src/canvas/content.ts +51 -2
- package/src/canvas/edit.ts +49 -6
- package/src/canvas/editing.ts +203 -0
- package/src/canvas/global.d.ts +15 -0
- package/src/canvas/server.ts +149 -12
- package/src/canvas/tokens.ts +60 -18
- package/src/census/count.ts +1 -1
- package/src/claude/seeds.ts +3 -3
- package/src/claude/skills-audit.ts +28 -0
- package/src/claude/skills-drift.ts +1 -1
- package/src/claude/skills-families.ts +36 -0
- package/src/claude/skills-list.ts +76 -15
- package/src/claude/skills-reach.ts +1 -1
- package/src/commands/audits.ts +2 -2
- package/src/commands/canvas.ts +180 -3
- package/src/commands/claude.ts +70 -12
- package/src/commands/design.ts +145 -5
- package/src/commands/docs.ts +1 -1
- package/src/commands/feedback.ts +2 -2
- package/src/commands/gate.ts +9 -2
- package/src/commands/gov.ts +3 -3
- package/src/commands/hooks.ts +1 -1
- package/src/commands/indexes.ts +1 -1
- package/src/commands/init.ts +2 -2
- package/src/commands/intake.ts +1 -1
- package/src/commands/migrate.ts +2 -2
- package/src/commands/pr.ts +288 -65
- package/src/commands/records.ts +1 -1
- package/src/commands/repo.ts +1 -1
- package/src/commands/sandbox.ts +64 -15
- package/src/commands/sessions.ts +1 -1
- package/src/commands/slides.ts +1 -1
- package/src/commands/sync.ts +2 -2
- package/src/commands/tasks.ts +21 -4
- package/src/commands/teach.ts +1 -1
- package/src/commands/tooling.ts +1 -1
- package/src/commands/transcripts.ts +2 -2
- package/src/commands/upgrade.ts +1 -1
- package/src/commands/wiki.ts +1 -1
- package/src/commands/worktrees.ts +1 -1
- package/src/comments/scan.ts +1 -1
- package/src/comments/trend.ts +1 -1
- package/src/context/architecture.ts +1 -1
- package/src/context/citations.ts +3 -3
- package/src/context/classify/extract.ts +3 -3
- package/src/context/folders.ts +2 -2
- package/src/context/requirements.ts +1 -1
- package/src/design/project-fonts.ts +392 -0
- package/src/docs/read.ts +1 -1
- package/src/exec.ts +1 -1
- package/src/feedback/github.ts +2 -2
- package/src/gate/measures.ts +48 -10
- package/src/gate/sequencer.ts +56 -4
- package/src/gate/stages.ts +17 -0
- package/src/gate/suite-lock.ts +214 -0
- package/src/{git-files.ts → git/files.ts} +2 -2
- package/src/{git-ignore.ts → git/ignore.ts} +1 -1
- package/src/{worktree.ts → git/worktree.ts} +1 -1
- package/src/gov/citations.ts +1 -1
- package/src/gov/install.ts +1 -1
- package/src/gov/superseded.ts +1 -1
- package/src/gov/test-order.ts +2 -2
- package/src/help.ts +1 -1
- package/src/hooks/post-merge.ts +1 -1
- package/src/indexes/walk.ts +1 -1
- package/src/intake/folder.ts +1 -1
- package/src/labels/audit.ts +1 -1
- package/src/labels/map.ts +1 -1
- package/src/labels/phase.ts +3 -3
- package/src/markdown/files.ts +1 -1
- package/src/migrate/apply.ts +1 -1
- package/src/migrate/record-layout.ts +1 -1
- package/src/migrate/records.ts +11 -11
- package/src/migrate/scratch-evidence.ts +1 -1
- package/src/migrate/surface-roots.ts +6 -6
- package/src/pr/bijection.ts +1 -1
- package/src/pr/evidence.ts +250 -59
- package/src/pr/local.ts +16 -10
- package/src/pr/preview.ts +78 -2
- package/src/pr/tick.ts +56 -0
- package/src/process/harness.ts +1 -1
- package/src/records/backup.ts +2 -2
- package/src/records/ordinal.ts +1 -1
- package/src/records/prune.ts +1 -1
- package/src/records/size.ts +1 -1
- package/src/records/stale-canonical.ts +2 -2
- package/src/records/stale.ts +1 -1
- package/src/records/validate.ts +2 -2
- package/src/{project-root.ts → roots/project.ts} +1 -1
- package/src/{record-root.ts → roots/record.ts} +1 -1
- package/src/sandbox/coverage.ts +11 -12
- package/src/sandbox/equivalence.ts +57 -26
- package/src/sandbox/headless/dispatch.ts +264 -0
- package/src/sandbox/headless/run.ts +653 -0
- package/src/sandbox/headless/watch.ts +145 -0
- package/src/sandbox/hooks.ts +1 -1
- package/src/sandbox/provision.ts +90 -29
- package/src/sandbox/scenario.ts +588 -0
- package/src/sandbox/tree.ts +4 -8
- package/src/secrets/scan.ts +1 -1
- package/src/sessions/claim.ts +1 -1
- package/src/sessions/live.ts +4 -1
- package/src/sessions/resolve.ts +1 -1
- package/src/sessions/transfer/bundle.ts +2 -2
- package/src/slides/decks.ts +1 -1
- package/src/slides/package/fonts.ts +6 -2
- package/src/standards/audit.ts +2 -2
- package/src/standards/read.ts +1 -1
- package/src/sync/check.ts +2 -2
- package/src/sync/engine.ts +3 -3
- package/src/sync/git.ts +1 -1
- package/src/sync/history.ts +1 -1
- package/src/sync/layout.ts +2 -2
- package/src/sync/reverse.ts +1 -1
- package/src/sync/stamp.ts +1 -1
- package/src/sync/target.ts +1 -1
- package/src/sync/workflow.ts +1 -1
- package/src/targets/pulls.ts +2 -2
- package/src/targets/sweep.ts +2 -2
- package/src/{target.ts → targets/validate.ts} +1 -1
- package/src/tasks/answers.ts +1 -1
- package/src/tasks/archive.ts +1 -1
- package/src/tasks/label.ts +1 -1
- package/src/tasks/reach.ts +2 -2
- package/src/tasks/trunk.ts +1 -1
- package/src/tasks/validate.ts +38 -14
- package/src/teach/nav.ts +1 -1
- package/src/teach/workspace.ts +1 -1
- package/src/tooling/inject.ts +2 -2
- package/src/tooling/read.ts +1 -1
- package/src/tooling/scan.ts +1 -1
- package/src/tooling/subfolder.ts +1 -1
- package/src/version/installed.ts +1 -1
- package/src/version/skew.ts +1 -1
- package/src/web/readme-citations.ts +1 -1
- package/src/worktrees/bare-flag.ts +1 -1
- package/src/worktrees/reclaim.ts +2 -2
- package/src/worktrees/remove.ts +1 -1
- package/standards/board.md +3 -3
- package/standards/canvas.md +2 -1
- package/standards/plan.md +13 -7
- package/standards/skill.md +1 -0
- package/standards/wiki.md +12 -3
- package/tooling/base/configs/.github/workflows/pr-local-preview.yml +2 -2
- package/tsconfig.json +1 -1
- package/scripts/lib/sandbox-dispatch.sh +0 -190
- /package/src/{client-commands.ts → gate/client-commands.ts} +0 -0
- /package/src/{git-base.ts → git/base.ts} +0 -0
- /package/src/{git-env.ts → git/env.ts} +0 -0
- /package/src/{legacy-path.ts → roots/legacy.ts} +0 -0
- /package/src/{surface-root.ts → roots/surface.ts} +0 -0
- /package/src/{copy.ts → sync/copy.ts} +0 -0
package/standards/wiki.md
CHANGED
|
@@ -47,9 +47,18 @@ A page failing either is non-conforming even when it satisfies every shape rule
|
|
|
47
47
|
## Sourcing
|
|
48
48
|
|
|
49
49
|
- Close the intro paragraph with a `Source:` sentence naming the owner. Link the canonical page where one exists, and name the owner alone where the subject has no single URL.
|
|
50
|
+
- Link a docs page in its fetchable form, `https://code.claude.com/docs/en/<page>.md`, so a session reads the live text rather than a copy.
|
|
50
51
|
- Fetch current information through the `claude-code-guide` agent when the subject is Claude Code. Do not work from training knowledge.
|
|
51
52
|
- Propose an addition or correction and wait for confirmation. Do not write to a wiki file unasked.
|
|
52
53
|
|
|
54
|
+
## Shape
|
|
55
|
+
|
|
56
|
+
- Keep a page thin: the `Source:` sentence, an orientation paragraph, and lesson sections. The source owns the reference detail and a paraphrase of it falls behind within days.
|
|
57
|
+
- Apply the lesson test per section: keep a fact about the subject only when the source page does not state it. Such a fact is a gotcha this project hit, a constraint it relies on, or how it uses the feature.
|
|
58
|
+
- Read the current source before keeping or cutting a section. A fact kept from memory is the drift this shape exists to prevent.
|
|
59
|
+
- Send a session that wants reference detail to the source and fetch it live. The page answers orientation and lessons only.
|
|
60
|
+
- Leave a page with no lessons as the source link and the orientation paragraph. A catalog routes a reader by page, so the stub stays.
|
|
61
|
+
|
|
53
62
|
## Template
|
|
54
63
|
|
|
55
64
|
```markdown
|
|
@@ -60,9 +69,9 @@ description: <one line naming what this page covers>
|
|
|
60
69
|
|
|
61
70
|
# <Subject>
|
|
62
71
|
|
|
63
|
-
<What the subject is and why it matters.> Source: <owner, with a link to the canonical page where one exists>.
|
|
72
|
+
<What the subject is and why it matters, in a short orientation.> Source: <owner, with a link to the canonical page where one exists>.
|
|
64
73
|
|
|
65
|
-
## <
|
|
74
|
+
## <Lesson>
|
|
66
75
|
|
|
67
|
-
<
|
|
76
|
+
<A fact the source page does not state.>
|
|
68
77
|
```
|
|
@@ -16,7 +16,7 @@ jobs:
|
|
|
16
16
|
# Say so and pass rather than failing a close nobody can fix.
|
|
17
17
|
- name: Skip a fork pull request
|
|
18
18
|
if: github.event.pull_request.head.repo.full_name != github.repository
|
|
19
|
-
run: echo "::notice::Fork pull request, so the token cannot edit the evidence comment. The
|
|
19
|
+
run: echo "::notice::Fork pull request, so the token cannot edit the evidence comment. The preview line stays."
|
|
20
20
|
|
|
21
21
|
- name: Setup Bun
|
|
22
22
|
if: github.event.pull_request.head.repo.full_name == github.repository
|
|
@@ -24,7 +24,7 @@ jobs:
|
|
|
24
24
|
with:
|
|
25
25
|
bun-version: latest
|
|
26
26
|
|
|
27
|
-
- name:
|
|
27
|
+
- name: Drop the preview line from the evidence comment
|
|
28
28
|
if: github.event.pull_request.head.repo.full_name == github.repository
|
|
29
29
|
env:
|
|
30
30
|
GH_TOKEN: ${{ github.token }}
|
package/tsconfig.json
CHANGED
|
@@ -1,190 +0,0 @@
|
|
|
1
|
-
#!/usr/bin/env bash
|
|
2
|
-
|
|
3
|
-
# The bound on a nested background dispatch, and the recording behind it.
|
|
4
|
-
#
|
|
5
|
-
# An arm invoked without the narration its fixture states dispatched a real
|
|
6
|
-
# `claude --bg` session against the machine's own process table, and nothing in
|
|
7
|
-
# the harness noticed. `snapshot_tree` reads the sandbox and `snapshot_root`
|
|
8
|
-
# reads four scratch directories, and a dispatched session writes into none of
|
|
9
|
-
# them, so the escape watch reported clean throughout and was right to. The run
|
|
10
|
-
# spent real cost until a person found it, and `SIGTERM` alone did not end it.
|
|
11
|
-
#
|
|
12
|
-
# Three mechanisms sit here rather than in `sandbox/run.sh`, which is the
|
|
13
|
-
# only caller. Each is reachable from a test that never launches a session:
|
|
14
|
-
# `src/sandbox/dispatch.test.ts` drives the shim against a stub binary and the
|
|
15
|
-
# reap against a `sleep`, where proving either through `run.sh` would mean
|
|
16
|
-
# spawning the thing the bound exists to prevent.
|
|
17
|
-
|
|
18
|
-
# The prevention. Writes a `claude` that refuses a background dispatch by flag
|
|
19
|
-
# name and delegates everything else, for `run.sh` to place first on the PATH of
|
|
20
|
-
# the session it spawns.
|
|
21
|
-
#
|
|
22
|
-
# The real binary is baked into the file rather than passed through the
|
|
23
|
-
# environment. An env var carrying it would sit in the spawned session's own
|
|
24
|
-
# environment, which hands any arm the exact string that walks around the shim.
|
|
25
|
-
#
|
|
26
|
-
# It names itself in its refusal. No arm shells to `claude` today, and one
|
|
27
|
-
# written tomorrow would meet this and read it as the harness being broken
|
|
28
|
-
# unless the text says which layer refused and why.
|
|
29
|
-
#
|
|
30
|
-
# It is not the whole bound. A dispatch reaching the binary by an absolute path
|
|
31
|
-
# never resolves through PATH at all, which is what `reap_process_group`
|
|
32
|
-
# backstops.
|
|
33
|
-
install_dispatch_shim() {
|
|
34
|
-
local dir="$1"
|
|
35
|
-
local real="$2"
|
|
36
|
-
|
|
37
|
-
cat >"$dir/claude" <<SHIM
|
|
38
|
-
#!/usr/bin/env bash
|
|
39
|
-
# Generated by install_dispatch_shim in scripts/lib/sandbox-dispatch.sh.
|
|
40
|
-
for arg in "\$@"; do
|
|
41
|
-
case "\$arg" in
|
|
42
|
-
--bg | --background)
|
|
43
|
-
echo "canon sandbox harness: refusing \$arg. A sandbox run may not dispatch a background session, so this shim stands first on PATH in place of the real claude binary. Drive the refusal the fixture asks for, or run this outside the harness." >&2
|
|
44
|
-
exit 64
|
|
45
|
-
;;
|
|
46
|
-
esac
|
|
47
|
-
done
|
|
48
|
-
exec "$real" "\$@"
|
|
49
|
-
SHIM
|
|
50
|
-
|
|
51
|
-
chmod +x "$dir/claude"
|
|
52
|
-
}
|
|
53
|
-
|
|
54
|
-
# True while any process still carries this group id. Reads the whole table
|
|
55
|
-
# rather than `ps -g`, which selects by session on some builds and reports an
|
|
56
|
-
# empty group as a clean exit either way.
|
|
57
|
-
process_group_alive() {
|
|
58
|
-
ps -eo pgid= 2>/dev/null | tr -d ' ' | grep -qx "$1"
|
|
59
|
-
}
|
|
60
|
-
|
|
61
|
-
# The backstop. Signals a process group `run.sh` created, escalating rather than
|
|
62
|
-
# sending one signal and reporting success, since the dispatch that produced this
|
|
63
|
-
# row took two signals to reap.
|
|
64
|
-
#
|
|
65
|
-
# Prints its outcome and always exits zero. Every caller runs under `set -e`, and
|
|
66
|
-
# a survivor is something to report rather than a reason to abandon the verdict
|
|
67
|
-
# the run already took.
|
|
68
|
-
#
|
|
69
|
-
# The group has to be one the run created, and the refusal below is what makes
|
|
70
|
-
# that property travel with the function rather than living in one caller.
|
|
71
|
-
# `run.sh` reads the harness group once and reports `inherited` when the session
|
|
72
|
-
# failed to lead a group of its own, which is a diagnosis; this is the floor
|
|
73
|
-
# under it, and under every caller after it, because the cost of getting the
|
|
74
|
-
# argument wrong is the operator's own shell.
|
|
75
|
-
#
|
|
76
|
-
# What it cannot reach is a dispatch that called `setsid` on its way out, which
|
|
77
|
-
# leaves the group before this reads it. That is the half the shim covers.
|
|
78
|
-
reap_process_group() {
|
|
79
|
-
local pgid="$1"
|
|
80
|
-
|
|
81
|
-
if [ "$pgid" = "$(ps -o pgid= -p $$ 2>/dev/null | tr -d ' ')" ]; then
|
|
82
|
-
printf 'refused-own-group\n'
|
|
83
|
-
return 0
|
|
84
|
-
fi
|
|
85
|
-
|
|
86
|
-
if ! process_group_alive "$pgid"; then
|
|
87
|
-
printf 'clear\n'
|
|
88
|
-
return 0
|
|
89
|
-
fi
|
|
90
|
-
|
|
91
|
-
kill -TERM -"$pgid" 2>/dev/null || true
|
|
92
|
-
wait_for_group_exit "$pgid"
|
|
93
|
-
|
|
94
|
-
if ! process_group_alive "$pgid"; then
|
|
95
|
-
printf 'reaped-term\n'
|
|
96
|
-
return 0
|
|
97
|
-
fi
|
|
98
|
-
|
|
99
|
-
kill -KILL -"$pgid" 2>/dev/null || true
|
|
100
|
-
wait_for_group_exit "$pgid"
|
|
101
|
-
|
|
102
|
-
if process_group_alive "$pgid"; then
|
|
103
|
-
printf 'survived\n'
|
|
104
|
-
else
|
|
105
|
-
printf 'reaped-kill\n'
|
|
106
|
-
fi
|
|
107
|
-
return 0
|
|
108
|
-
}
|
|
109
|
-
|
|
110
|
-
# Five seconds per signal. A shorter window reports a survivor for a process
|
|
111
|
-
# that was already exiting, and a longer one stalls a run whose verdict is
|
|
112
|
-
# already taken.
|
|
113
|
-
wait_for_group_exit() {
|
|
114
|
-
local pgid="$1"
|
|
115
|
-
local waited=0
|
|
116
|
-
|
|
117
|
-
while [ "$waited" -lt 20 ] && process_group_alive "$pgid"; do
|
|
118
|
-
sleep 0.25
|
|
119
|
-
waited=$((waited + 1))
|
|
120
|
-
done
|
|
121
|
-
}
|
|
122
|
-
|
|
123
|
-
# The client's own session registry, which is the only surface a nested dispatch
|
|
124
|
-
# leaves a trace on that this harness can read. It is client-owned and carries no
|
|
125
|
-
# contract, so a client that stops writing a record per session breaks the
|
|
126
|
-
# detection silently. That is why what reads it reports rather than asserts.
|
|
127
|
-
sessions_dir() {
|
|
128
|
-
printf '%s/sessions\n' "${CLAUDE_CONFIG_DIR:-$HOME/.claude}"
|
|
129
|
-
}
|
|
130
|
-
|
|
131
|
-
# Set whenever the registry directory existed at snapshot time. An absent
|
|
132
|
-
# registry produces the same empty manifest a quiet run does, and the two mean
|
|
133
|
-
# opposite things, so the report carries this rather than letting an unwatched
|
|
134
|
-
# run read as a clean one. Same distinction `escape_watched` draws in `run.sh`,
|
|
135
|
-
# which is also the only reader, hence the exemption on the write below.
|
|
136
|
-
sessions_watched=0
|
|
137
|
-
|
|
138
|
-
# Record names only, never hashes. A live session rewrites its own record on
|
|
139
|
-
# every status change, so a content manifest would name every session running
|
|
140
|
-
# beside this one, which is the noise the escape watch was already narrowed to
|
|
141
|
-
# avoid. A record that appears between the two snapshots is a session that
|
|
142
|
-
# started during the run, and that is the whole question.
|
|
143
|
-
snapshot_sessions() {
|
|
144
|
-
local manifest="$1"
|
|
145
|
-
local dir
|
|
146
|
-
dir="$(sessions_dir)"
|
|
147
|
-
|
|
148
|
-
: >"$manifest"
|
|
149
|
-
[ -d "$dir" ] || return 0
|
|
150
|
-
# shellcheck disable=SC2034
|
|
151
|
-
sessions_watched=1
|
|
152
|
-
|
|
153
|
-
find "$dir" -maxdepth 1 -type f -name '*.json' 2>/dev/null |
|
|
154
|
-
sed 's|.*/||' | sort >"$manifest"
|
|
155
|
-
return 0
|
|
156
|
-
}
|
|
157
|
-
|
|
158
|
-
# The new side alone, unlike `writes_between`. A record that vanished is a
|
|
159
|
-
# session that ended and a record that changed is one that was already running,
|
|
160
|
-
# and neither is a dispatch this run made.
|
|
161
|
-
sessions_between() {
|
|
162
|
-
comm -13 "$1" "$2"
|
|
163
|
-
}
|
|
164
|
-
|
|
165
|
-
# The side `sessions_between` drops. A record present on both sides is a session
|
|
166
|
-
# that was already running before this run started and still was after, which
|
|
167
|
-
# rules it out as this run's own dispatch and rules it in as a witness able to
|
|
168
|
-
# explain a write the escape watch attributes to nobody.
|
|
169
|
-
sessions_concurrent() {
|
|
170
|
-
comm -12 "$1" "$2"
|
|
171
|
-
}
|
|
172
|
-
|
|
173
|
-
# Turns a record name into a line a person can act on. The file can be gone by
|
|
174
|
-
# the time this reads it, since a session that started and exited inside the run
|
|
175
|
-
# takes its record with it, so an unreadable record still reports its name rather
|
|
176
|
-
# than dropping out of the list.
|
|
177
|
-
#
|
|
178
|
-
# One record is one line. The name is written by whatever peer claimed the
|
|
179
|
-
# session, so a newline inside it would split one record across two entries in a
|
|
180
|
-
# list every reader takes as one line each.
|
|
181
|
-
describe_session() {
|
|
182
|
-
local record desc
|
|
183
|
-
record="$(sessions_dir)/$1"
|
|
184
|
-
|
|
185
|
-
desc="$(jq -r '"\(.name // "unnamed") in \(.cwd // "an unrecorded directory")"
|
|
186
|
-
| gsub("\\s+"; " ")' "$record" 2>/dev/null || true)"
|
|
187
|
-
[ -z "$desc" ] && desc="record already gone"
|
|
188
|
-
|
|
189
|
-
printf '%s: %s\n' "$1" "$desc"
|
|
190
|
-
}
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|
|
File without changes
|