claude-dev-env 2.3.0 → 2.4.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/CLAUDE.md +3 -2
- package/agents/CLAUDE.md +1 -0
- package/agents/code-verifier.md +3 -3
- package/agents/skill-writer-agent.md +84 -0
- package/audit-rubrics/category_rubrics/category-o-docstring-vs-impl-drift.md +141 -41
- package/audit-rubrics/prompts/category-o-docstring-vs-impl-drift.md +29 -13
- package/docs/CLAUDE.md +1 -0
- package/docs/references/CLAUDE.md +1 -0
- package/docs/references/code-review-enforcement.md +97 -0
- package/docs/wsl-docker-cowork-starter-matrix.md +89 -0
- package/hooks/blocking/CLAUDE.md +8 -1
- package/hooks/blocking/code_review_enforcement_config_bootstrap.py +53 -0
- package/hooks/blocking/code_review_gate_deny.py +74 -0
- package/hooks/blocking/code_review_pr_create_gate.py +194 -0
- package/hooks/blocking/code_review_push_gate.py +140 -0
- package/hooks/blocking/code_review_stamp_directory_write_blocker.py +340 -0
- package/hooks/blocking/code_review_stamp_store.py +233 -0
- package/hooks/blocking/code_review_stamp_write_blocker_parts/__init__.py +7 -0
- package/hooks/blocking/code_review_stamp_write_blocker_parts/conftest.py +15 -0
- package/hooks/blocking/code_review_stamp_write_blocker_parts/obfuscated_stamp_path_reference.py +212 -0
- package/hooks/blocking/code_review_stamp_write_blocker_parts/split_directory_change_into_stamp.py +138 -0
- package/hooks/blocking/code_review_stamp_write_blocker_parts/test_obfuscated_stamp_path_reference.py +49 -0
- package/hooks/blocking/code_review_stamp_write_blocker_parts/test_split_directory_change_into_stamp.py +38 -0
- package/hooks/blocking/code_verifier_spawn_preflight_gate.py +39 -27
- package/hooks/blocking/config/code_review_enforcement_constants.py +110 -0
- package/hooks/blocking/config/test_code_review_enforcement_constants.py +108 -0
- package/hooks/blocking/config/verified_commit_constants.py +13 -9
- package/hooks/blocking/conftest.py +2 -0
- package/hooks/blocking/convergence_gate_blocker.py +112 -23
- package/hooks/blocking/destructive_command_blocker.py +19 -6
- package/hooks/blocking/pr_description_proof_of_work.py +52 -34
- package/hooks/blocking/test_bash_pre_tool_use_dispatcher.py +4 -1
- package/hooks/blocking/test_code_review_enforcement_config_bootstrap.py +62 -0
- package/hooks/blocking/test_code_review_gate_deny.py +54 -0
- package/hooks/blocking/test_code_review_pr_create_gate.py +185 -0
- package/hooks/blocking/test_code_review_push_gate.py +189 -0
- package/hooks/blocking/test_code_review_stamp_directory_write_blocker.py +180 -0
- package/hooks/blocking/test_code_review_stamp_store.py +205 -0
- package/hooks/blocking/test_code_verifier_spawn_preflight_gate.py +124 -2
- package/hooks/blocking/test_convergence_gate_blocker.py +153 -5
- package/hooks/blocking/test_destructive_command_blocker.py +1 -1
- package/hooks/blocking/test_destructive_command_blocker_deny_mode.py +45 -0
- package/hooks/blocking/test_pr_description_proof_of_work.py +151 -0
- package/hooks/blocking/test_pre_tool_use_dispatcher.py +8 -8
- package/hooks/blocking/test_verification_verdict_store.py +920 -903
- package/hooks/blocking/test_volatile_path_in_post_blocker.py +114 -2
- package/hooks/blocking/verification_verdict_store.py +27 -5
- package/hooks/blocking/verified_commit_gate_parts/gated_invocations.py +29 -17
- package/hooks/blocking/verified_commit_gate_parts/tests/test_gated_invocations.py +35 -0
- package/hooks/blocking/volatile_path_in_post_blocker.py +69 -8
- package/hooks/git-hooks/git_hooks_constants/__init__.py +6 -0
- package/hooks/git-hooks/pre_push.py +89 -2
- package/hooks/git-hooks/test_pre_push.py +103 -0
- package/hooks/hooks.json +16 -1
- package/hooks/hooks_constants/CLAUDE.md +1 -0
- package/hooks/hooks_constants/bash_pre_tool_use_dispatcher_constants.py +8 -0
- package/hooks/hooks_constants/code_rules_path_utils_constants.py +1 -0
- package/hooks/hooks_constants/code_verifier_spawn_preflight_gate_constants.py +26 -11
- package/hooks/hooks_constants/convergence_gate_blocker_constants.py +20 -3
- package/hooks/hooks_constants/destructive_command_segment_constants.py +3 -1
- package/hooks/hooks_constants/pr_description_proof_of_work_constants.py +0 -4
- package/hooks/hooks_constants/pre_tool_use_dispatcher_constants.py +4 -0
- package/hooks/hooks_constants/pyproject_config_discovery_constants.py +16 -0
- package/hooks/hooks_constants/test_bash_pre_tool_use_dispatcher_constants.py +24 -0
- package/hooks/hooks_constants/test_pre_tool_use_dispatcher_constants.py +6 -0
- package/hooks/hooks_constants/volatile_path_in_post_blocker_constants.py +8 -1
- package/hooks/validators/CLAUDE.md +1 -0
- package/hooks/validators/mypy_integration.py +63 -50
- package/hooks/validators/pyproject_config_discovery.py +101 -0
- package/hooks/validators/ruff_integration.py +211 -23
- package/hooks/validators/run_all_validators.py +21 -14
- package/hooks/validators/test_mypy_integration.py +32 -0
- package/hooks/validators/test_pyproject_config_discovery.py +94 -0
- package/hooks/validators/test_ruff_integration.py +68 -1
- package/hooks/validators/test_run_all_validators.py +25 -0
- package/hooks/validators/test_run_all_validators_config_discovery.py +123 -0
- package/package.json +1 -1
- package/rules/docstring-prose-matches-implementation.md +20 -43
- package/rules/durable-post-artifacts.md +7 -0
- package/scripts/codec_forwarding_test_support.py +83 -0
- package/scripts/conftest.py +8 -0
- package/scripts/dev_env_scripts_constants/CLAUDE.md +1 -1
- package/scripts/dev_env_scripts_constants/claude_chain_constants.py +44 -1
- package/scripts/dev_env_scripts_constants/code_review_constants.py +129 -12
- package/scripts/dev_env_scripts_constants/test_code_review_constants.py +55 -0
- package/scripts/invoke_code_review.py +550 -38
- package/scripts/resolve_worker_spawn.py +8 -1
- package/scripts/test_invoke_code_review.py +298 -4
- package/scripts/test_invoke_code_review_codec.py +77 -0
- package/scripts/test_resolve_worker_spawn_codec.py +101 -0
- package/skills/autoconverge/SKILL.md +9 -3
- package/skills/autoconverge/reference/convergence.md +2 -1
- package/skills/autoconverge/reference/multi-pr.md +6 -1
- package/skills/autoconverge/reference/stop-conditions.md +16 -10
- package/skills/autoconverge/workflow/converge.clean-audit.test.mjs +1 -1
- package/skills/autoconverge/workflow/converge.codex-gate.test.mjs +175 -3
- package/skills/autoconverge/workflow/converge.contract.test.mjs +19 -0
- package/skills/autoconverge/workflow/converge.mjs +24 -8
- package/skills/autoconverge/workflow/converge_multi.mjs +7 -3
- package/skills/autoconverge/workflow/converge_multi.run-input.test.mjs +5 -0
- package/skills/fresh-branch/CLAUDE.md +2 -0
- package/skills/fresh-branch/SKILL.md +2 -0
- package/skills/fresh-branch/scripts/create_fresh_branch.py +78 -180
- package/skills/fresh-branch/scripts/fresh_branch_git_commands.py +285 -0
- package/skills/fresh-branch/scripts/fresh_branch_scripts_constants/fresh_branch_cli_constants.py +1 -0
- package/skills/fresh-branch/scripts/pytest.ini +4 -0
- package/skills/fresh-branch/scripts/test_create_fresh_branch.py +98 -0
- package/skills/fresh-branch/scripts/test_fresh_branch_git_commands.py +310 -0
- package/skills/prototype/SKILL.md +86 -0
- package/skills/prototype/reference/honest-limitations.md +23 -0
- package/skills/prototype/reference/promotion-tasks.md +23 -0
- package/skills/prototype/scripts/build_sandbox_settings.py +249 -0
- package/skills/prototype/scripts/conftest.py +15 -0
- package/skills/prototype/scripts/launch_sandbox.py +205 -0
- package/skills/prototype/scripts/probe_sandbox_safety.py +311 -0
- package/skills/prototype/scripts/prototype_scripts_constants/__init__.py +1 -0
- package/skills/prototype/scripts/prototype_scripts_constants/config/__init__.py +0 -0
- package/skills/prototype/scripts/prototype_scripts_constants/config/build_sandbox_settings_constants.py +41 -0
- package/skills/prototype/scripts/prototype_scripts_constants/config/launch_sandbox_constants.py +23 -0
- package/skills/prototype/scripts/prototype_scripts_constants/config/probe_sandbox_safety_constants.py +45 -0
- package/skills/prototype/scripts/prototype_scripts_constants/config/prototype_common_constants.py +10 -0
- package/skills/prototype/scripts/test_build_sandbox_settings.py +275 -0
- package/skills/prototype/scripts/test_launch_sandbox.py +303 -0
- package/skills/prototype/scripts/test_probe_sandbox_safety.py +284 -0
- package/skills/prototype/workflows/promotion.md +27 -0
- package/skills/prototype/workflows/sandbox.md +35 -0
- package/skills/skill-builder/CLAUDE.md +3 -3
- package/skills/skill-builder/SKILL.md +5 -5
- package/skills/skill-builder/references/CLAUDE.md +1 -1
- package/skills/skill-builder/references/delegation-map.md +3 -3
- package/skills/skill-builder/references/description-field.md +1 -1
- package/skills/skill-builder/references/skill-modularity.md +2 -3
- package/skills/skill-builder/workflows/CLAUDE.md +1 -1
- package/skills/skill-builder/workflows/improve-skill.md +1 -1
- package/skills/skill-builder/workflows/new-skill.md +2 -2
- package/skills/team-advisor/SKILL.md +2 -2
|
@@ -0,0 +1,249 @@
|
|
|
1
|
+
#!/usr/bin/env python3
|
|
2
|
+
"""Emit a minimal ``--settings`` file carrying only the two sandbox safety hooks.
|
|
3
|
+
|
|
4
|
+
::
|
|
5
|
+
|
|
6
|
+
python build_sandbox_settings.py --out sandbox-settings.json
|
|
7
|
+
sandbox-settings.json -> {"env": {deny-mode override},
|
|
8
|
+
"hooks": {"PreToolUse": [
|
|
9
|
+
{"matcher": "Bash", "hooks": [pii_prevention, destructive_command]},
|
|
10
|
+
{"matcher": "Edit", "hooks": [pii_prevention]},
|
|
11
|
+
{"matcher": "MultiEdit", "hooks": [pii_prevention]},
|
|
12
|
+
{"matcher": "Write", "hooks": [pii_prevention]}]}}
|
|
13
|
+
|
|
14
|
+
The builder resolves each safety hook's command entry from the live settings
|
|
15
|
+
source, which carries the per-machine interpreter, absolute script path, and
|
|
16
|
+
timeout. It then registers each entry on the matchers the sandbox requires.
|
|
17
|
+
The personal-data gate covers every write surface and the command line. The
|
|
18
|
+
destructive-command gate covers the command line, and the emitted ``env`` block
|
|
19
|
+
runs that gate in deny mode so its block holds under
|
|
20
|
+
``--dangerously-skip-permissions``. When either safety hook is absent from the
|
|
21
|
+
source, the builder exits without writing, because a sandbox cannot be
|
|
22
|
+
contained without both.
|
|
23
|
+
"""
|
|
24
|
+
|
|
25
|
+
from __future__ import annotations
|
|
26
|
+
|
|
27
|
+
import argparse
|
|
28
|
+
import json
|
|
29
|
+
import logging
|
|
30
|
+
import sys
|
|
31
|
+
from pathlib import Path
|
|
32
|
+
|
|
33
|
+
from prototype_scripts_constants.config.build_sandbox_settings_constants import (
|
|
34
|
+
ALL_REQUIRED_MATCHERS_BY_SAFETY_BASENAME,
|
|
35
|
+
ALL_SAFETY_HOOK_SCRIPT_BASENAMES,
|
|
36
|
+
BUILD_SUCCESS_EXIT_CODE,
|
|
37
|
+
COMMAND_KEY,
|
|
38
|
+
DEFAULT_SETTINGS_SOURCE,
|
|
39
|
+
DESTRUCTIVE_DENY_MODE_ENV_VALUE,
|
|
40
|
+
DESTRUCTIVE_DENY_MODE_ENV_VAR,
|
|
41
|
+
ENV_KEY,
|
|
42
|
+
HOOKS_KEY,
|
|
43
|
+
JSON_INDENT_SPACES,
|
|
44
|
+
MATCHER_KEY,
|
|
45
|
+
MISSING_BASENAMES_JOIN_SEPARATOR,
|
|
46
|
+
PRE_TOOL_USE_KEY,
|
|
47
|
+
SETTINGS_MISSING_SAFETY_HOOK_EXIT_CODE,
|
|
48
|
+
SETTINGS_SOURCE_UNREADABLE_EXIT_CODE,
|
|
49
|
+
)
|
|
50
|
+
from prototype_scripts_constants.config.prototype_common_constants import (
|
|
51
|
+
LOGGING_FORMAT,
|
|
52
|
+
TEXT_ENCODING_UTF8,
|
|
53
|
+
)
|
|
54
|
+
|
|
55
|
+
logger = logging.getLogger("build_sandbox_settings")
|
|
56
|
+
|
|
57
|
+
|
|
58
|
+
def read_settings_document(source_path: Path) -> dict:
|
|
59
|
+
"""Load the live settings JSON the sandbox settings are resolved from.
|
|
60
|
+
|
|
61
|
+
::
|
|
62
|
+
|
|
63
|
+
~/.claude/settings.json -> parsed dict with hooks.PreToolUse blocks
|
|
64
|
+
"""
|
|
65
|
+
return json.loads(source_path.read_text(encoding=TEXT_ENCODING_UTF8))
|
|
66
|
+
|
|
67
|
+
|
|
68
|
+
def _first_subhook_running(
|
|
69
|
+
pre_tool_use_block: dict, script_basename: str
|
|
70
|
+
) -> dict | None:
|
|
71
|
+
"""Return the block's first sub-hook whose command runs the named script."""
|
|
72
|
+
for each_subhook in pre_tool_use_block.get(HOOKS_KEY, []):
|
|
73
|
+
if script_basename in each_subhook.get(COMMAND_KEY, ""):
|
|
74
|
+
return each_subhook
|
|
75
|
+
return None
|
|
76
|
+
|
|
77
|
+
|
|
78
|
+
def _first_block_running(
|
|
79
|
+
all_pre_tool_use_blocks: list[dict], script_basename: str
|
|
80
|
+
) -> dict | None:
|
|
81
|
+
"""Return the first block's sub-hook entry that runs the named safety script."""
|
|
82
|
+
for each_block in all_pre_tool_use_blocks:
|
|
83
|
+
resolved_entry = _first_subhook_running(each_block, script_basename)
|
|
84
|
+
if resolved_entry is not None:
|
|
85
|
+
return resolved_entry
|
|
86
|
+
return None
|
|
87
|
+
|
|
88
|
+
|
|
89
|
+
def resolve_safety_hook_entries(settings_document: dict) -> dict[str, dict]:
|
|
90
|
+
"""Resolve each safety hook's live sub-hook entry, keyed by script basename.
|
|
91
|
+
|
|
92
|
+
::
|
|
93
|
+
|
|
94
|
+
live PreToolUse blocks
|
|
95
|
+
-> {"pii_prevention_blocker.py": {type, command, timeout},
|
|
96
|
+
"destructive_command_blocker.py": {type, command, timeout}}
|
|
97
|
+
|
|
98
|
+
The captured entry carries the per-machine interpreter, absolute script
|
|
99
|
+
path, and timeout, so the sandbox settings stay correct on any machine.
|
|
100
|
+
|
|
101
|
+
Args:
|
|
102
|
+
settings_document: the live settings JSON with hooks.PreToolUse blocks.
|
|
103
|
+
|
|
104
|
+
Returns:
|
|
105
|
+
The first matching sub-hook entry per safety basename; a basename with
|
|
106
|
+
no matching sub-hook is absent from the mapping.
|
|
107
|
+
"""
|
|
108
|
+
pre_tool_use_blocks = settings_document.get(HOOKS_KEY, {}).get(PRE_TOOL_USE_KEY, [])
|
|
109
|
+
entry_by_basename: dict[str, dict] = {}
|
|
110
|
+
for each_basename in ALL_SAFETY_HOOK_SCRIPT_BASENAMES:
|
|
111
|
+
resolved_entry = _first_block_running(pre_tool_use_blocks, each_basename)
|
|
112
|
+
if resolved_entry is not None:
|
|
113
|
+
entry_by_basename[each_basename] = resolved_entry
|
|
114
|
+
return entry_by_basename
|
|
115
|
+
|
|
116
|
+
|
|
117
|
+
def find_unresolved_safety_hook_basenames(
|
|
118
|
+
entry_by_basename: dict[str, dict],
|
|
119
|
+
) -> list[str]:
|
|
120
|
+
"""List the safety hook basenames the live source resolved no command for.
|
|
121
|
+
|
|
122
|
+
::
|
|
123
|
+
|
|
124
|
+
both resolved -> []
|
|
125
|
+
destructive unresolved -> ["destructive_command_blocker.py"]
|
|
126
|
+
|
|
127
|
+
Args:
|
|
128
|
+
entry_by_basename: the resolved sub-hook entry per safety basename.
|
|
129
|
+
|
|
130
|
+
Returns:
|
|
131
|
+
The safety hook basenames absent from the mapping, empty when both
|
|
132
|
+
resolved.
|
|
133
|
+
"""
|
|
134
|
+
return [
|
|
135
|
+
each_basename
|
|
136
|
+
for each_basename in ALL_SAFETY_HOOK_SCRIPT_BASENAMES
|
|
137
|
+
if each_basename not in entry_by_basename
|
|
138
|
+
]
|
|
139
|
+
|
|
140
|
+
|
|
141
|
+
def _build_pre_tool_use_blocks(entry_by_basename: dict[str, dict]) -> list[dict]:
|
|
142
|
+
"""Build one PreToolUse block per required matcher, in sorted matcher order."""
|
|
143
|
+
entries_by_matcher: dict[str, list[dict]] = {}
|
|
144
|
+
for each_basename in ALL_REQUIRED_MATCHERS_BY_SAFETY_BASENAME:
|
|
145
|
+
resolved_entry = entry_by_basename[each_basename]
|
|
146
|
+
for each_matcher in ALL_REQUIRED_MATCHERS_BY_SAFETY_BASENAME[each_basename]:
|
|
147
|
+
entries_by_matcher.setdefault(each_matcher, []).append(resolved_entry)
|
|
148
|
+
return [
|
|
149
|
+
{MATCHER_KEY: each_matcher, HOOKS_KEY: entries_by_matcher[each_matcher]}
|
|
150
|
+
for each_matcher in sorted(entries_by_matcher)
|
|
151
|
+
]
|
|
152
|
+
|
|
153
|
+
|
|
154
|
+
def _build_deny_mode_env_block() -> dict[str, str]:
|
|
155
|
+
"""Build the env block that runs the destructive gate in deny mode.
|
|
156
|
+
|
|
157
|
+
The single key is the deny-mode env variable, mapped to its truthy value,
|
|
158
|
+
so the sandbox session runs the destructive gate with a hard block.
|
|
159
|
+
|
|
160
|
+
Returns:
|
|
161
|
+
The single-key env override the sandbox session applies.
|
|
162
|
+
"""
|
|
163
|
+
return {DESTRUCTIVE_DENY_MODE_ENV_VAR: DESTRUCTIVE_DENY_MODE_ENV_VALUE}
|
|
164
|
+
|
|
165
|
+
|
|
166
|
+
def build_minimal_settings(entry_by_basename: dict[str, dict]) -> dict:
|
|
167
|
+
"""Assemble the minimal settings document from the resolved safety entries.
|
|
168
|
+
|
|
169
|
+
::
|
|
170
|
+
|
|
171
|
+
resolved entries -> {"env": {deny-mode override},
|
|
172
|
+
"hooks": {"PreToolUse": [required-matcher blocks]}}
|
|
173
|
+
|
|
174
|
+
The ``env`` block runs the destructive gate in deny mode, so its block
|
|
175
|
+
holds under ``--dangerously-skip-permissions``, where an ask decision is
|
|
176
|
+
auto-resolved.
|
|
177
|
+
|
|
178
|
+
Args:
|
|
179
|
+
entry_by_basename: the resolved sub-hook entry per safety basename.
|
|
180
|
+
|
|
181
|
+
Returns:
|
|
182
|
+
The minimal settings document registering each safety entry on the
|
|
183
|
+
matchers the sandbox requires, with the deny-mode env override.
|
|
184
|
+
"""
|
|
185
|
+
return {
|
|
186
|
+
ENV_KEY: _build_deny_mode_env_block(),
|
|
187
|
+
HOOKS_KEY: {PRE_TOOL_USE_KEY: _build_pre_tool_use_blocks(entry_by_basename)},
|
|
188
|
+
}
|
|
189
|
+
|
|
190
|
+
|
|
191
|
+
def write_minimal_settings(minimal_settings: dict, out_path: Path) -> None:
|
|
192
|
+
"""Write the minimal settings document to the output path as pretty JSON."""
|
|
193
|
+
serialized_settings = json.dumps(minimal_settings, indent=JSON_INDENT_SPACES)
|
|
194
|
+
out_path.write_text(serialized_settings + "\n", encoding=TEXT_ENCODING_UTF8)
|
|
195
|
+
|
|
196
|
+
|
|
197
|
+
def _parse_arguments(all_arguments: list[str] | None) -> argparse.Namespace:
|
|
198
|
+
parser = argparse.ArgumentParser(description=__doc__)
|
|
199
|
+
parser.add_argument("--out", required=True, help="where to write the settings")
|
|
200
|
+
parser.add_argument(
|
|
201
|
+
"--settings-source",
|
|
202
|
+
default=DEFAULT_SETTINGS_SOURCE,
|
|
203
|
+
help="the live settings JSON to resolve the safety hook commands from",
|
|
204
|
+
)
|
|
205
|
+
return parser.parse_args(all_arguments)
|
|
206
|
+
|
|
207
|
+
|
|
208
|
+
def _load_settings_or_none(source_path: Path) -> dict | None:
|
|
209
|
+
"""Read and parse the settings source, logging and returning None on failure."""
|
|
210
|
+
try:
|
|
211
|
+
return read_settings_document(source_path)
|
|
212
|
+
except (OSError, json.JSONDecodeError) as read_error:
|
|
213
|
+
logger.error("cannot read settings source %s: %s", source_path, read_error)
|
|
214
|
+
return None
|
|
215
|
+
|
|
216
|
+
|
|
217
|
+
def main(all_arguments: list[str] | None = None) -> int:
|
|
218
|
+
"""Build the minimal safety settings and write them to the output path.
|
|
219
|
+
|
|
220
|
+
Args:
|
|
221
|
+
all_arguments: the command-line arguments, or None to read sys.argv.
|
|
222
|
+
|
|
223
|
+
Returns:
|
|
224
|
+
0 after writing the settings, 2 when the source is unreadable, and 2
|
|
225
|
+
when either safety hook is unresolved from the source.
|
|
226
|
+
"""
|
|
227
|
+
logging.basicConfig(format=LOGGING_FORMAT)
|
|
228
|
+
arguments = _parse_arguments(all_arguments)
|
|
229
|
+
source_path = Path(arguments.settings_source).expanduser()
|
|
230
|
+
settings_document = _load_settings_or_none(source_path)
|
|
231
|
+
if settings_document is None:
|
|
232
|
+
return SETTINGS_SOURCE_UNREADABLE_EXIT_CODE
|
|
233
|
+
entry_by_basename = resolve_safety_hook_entries(settings_document)
|
|
234
|
+
all_unresolved_basenames = find_unresolved_safety_hook_basenames(entry_by_basename)
|
|
235
|
+
if all_unresolved_basenames:
|
|
236
|
+
logger.error(
|
|
237
|
+
"safety hook missing from settings source: %s",
|
|
238
|
+
MISSING_BASENAMES_JOIN_SEPARATOR.join(all_unresolved_basenames),
|
|
239
|
+
)
|
|
240
|
+
return SETTINGS_MISSING_SAFETY_HOOK_EXIT_CODE
|
|
241
|
+
minimal_settings = build_minimal_settings(entry_by_basename)
|
|
242
|
+
out_path = Path(arguments.out).expanduser()
|
|
243
|
+
write_minimal_settings(minimal_settings, out_path)
|
|
244
|
+
sys.stdout.write(str(out_path) + "\n")
|
|
245
|
+
return BUILD_SUCCESS_EXIT_CODE
|
|
246
|
+
|
|
247
|
+
|
|
248
|
+
if __name__ == "__main__":
|
|
249
|
+
raise SystemExit(main(sys.argv[1:]))
|
|
@@ -0,0 +1,15 @@
|
|
|
1
|
+
"""Put this worktree's prototype scripts directory on the import path.
|
|
2
|
+
|
|
3
|
+
The scripts import their ``prototype_scripts_constants`` package by name.
|
|
4
|
+
Inserting this directory lets the tests resolve that package and the
|
|
5
|
+
script modules from the local worktree rather than an installed copy.
|
|
6
|
+
"""
|
|
7
|
+
|
|
8
|
+
from __future__ import annotations
|
|
9
|
+
|
|
10
|
+
import sys
|
|
11
|
+
from pathlib import Path
|
|
12
|
+
|
|
13
|
+
SCRIPTS_DIRECTORY = Path(__file__).resolve().parent
|
|
14
|
+
if str(SCRIPTS_DIRECTORY) not in sys.path:
|
|
15
|
+
sys.path.insert(0, str(SCRIPTS_DIRECTORY))
|
|
@@ -0,0 +1,205 @@
|
|
|
1
|
+
#!/usr/bin/env python3
|
|
2
|
+
"""Launch the hookless ``claude -p --bare`` sandbox session in a worktree.
|
|
3
|
+
|
|
4
|
+
::
|
|
5
|
+
|
|
6
|
+
python launch_sandbox.py --worktree ./wt --settings ./s.json --task-file ./t.txt
|
|
7
|
+
{"worktree": "./wt", "settings": "./s.json", "exit_code": 0}
|
|
8
|
+
|
|
9
|
+
The launcher validates the three paths, reads the proof-of-concept task from
|
|
10
|
+
the task file, and runs a headless ``claude`` session in the worktree under
|
|
11
|
+
the minimal safety settings. The session runs with the standards gates
|
|
12
|
+
stripped, so the two safety hooks in the settings are its only containment.
|
|
13
|
+
"""
|
|
14
|
+
|
|
15
|
+
from __future__ import annotations
|
|
16
|
+
|
|
17
|
+
import argparse
|
|
18
|
+
import json
|
|
19
|
+
import logging
|
|
20
|
+
import subprocess
|
|
21
|
+
import sys
|
|
22
|
+
from collections.abc import Callable
|
|
23
|
+
from pathlib import Path
|
|
24
|
+
|
|
25
|
+
from prototype_scripts_constants.config.launch_sandbox_constants import (
|
|
26
|
+
BARE_FLAG,
|
|
27
|
+
CLAUDE_EXECUTABLE_NAME,
|
|
28
|
+
DEFAULT_TIMEOUT_SECONDS,
|
|
29
|
+
LAUNCH_MISSING_PATH_EXIT_CODE,
|
|
30
|
+
LAUNCH_TIMEOUT_EXIT_CODE,
|
|
31
|
+
PROMPT_FLAG,
|
|
32
|
+
SETTINGS_FLAG,
|
|
33
|
+
SKIP_PERMISSIONS_FLAG,
|
|
34
|
+
SUMMARY_KEY_EXIT_CODE,
|
|
35
|
+
SUMMARY_KEY_SETTINGS,
|
|
36
|
+
SUMMARY_KEY_WORKTREE,
|
|
37
|
+
)
|
|
38
|
+
from prototype_scripts_constants.config.prototype_common_constants import (
|
|
39
|
+
LOGGING_FORMAT,
|
|
40
|
+
TEXT_ENCODING_UTF8,
|
|
41
|
+
)
|
|
42
|
+
|
|
43
|
+
logger = logging.getLogger("launch_sandbox")
|
|
44
|
+
|
|
45
|
+
SandboxCommandRunner = Callable[[list[str], Path, "int | None"], int]
|
|
46
|
+
|
|
47
|
+
|
|
48
|
+
def build_sandbox_command(task_text: str, settings_path: Path) -> list[str]:
|
|
49
|
+
"""Build the exact headless ``claude`` argument vector for the sandbox.
|
|
50
|
+
|
|
51
|
+
::
|
|
52
|
+
|
|
53
|
+
"build a spike", settings.json
|
|
54
|
+
-> ["claude", "-p", "build a spike", "--bare",
|
|
55
|
+
"--dangerously-skip-permissions", "--settings", "settings.json"]
|
|
56
|
+
|
|
57
|
+
Args:
|
|
58
|
+
task_text: the proof-of-concept task the session builds.
|
|
59
|
+
settings_path: the minimal safety settings file.
|
|
60
|
+
|
|
61
|
+
Returns:
|
|
62
|
+
The ordered argument vector for the headless claude session.
|
|
63
|
+
"""
|
|
64
|
+
return [
|
|
65
|
+
CLAUDE_EXECUTABLE_NAME,
|
|
66
|
+
PROMPT_FLAG,
|
|
67
|
+
task_text,
|
|
68
|
+
BARE_FLAG,
|
|
69
|
+
SKIP_PERMISSIONS_FLAG,
|
|
70
|
+
SETTINGS_FLAG,
|
|
71
|
+
str(settings_path),
|
|
72
|
+
]
|
|
73
|
+
|
|
74
|
+
|
|
75
|
+
def validate_sandbox_paths(
|
|
76
|
+
worktree_path: Path, settings_path: Path, task_file_path: Path
|
|
77
|
+
) -> str | None:
|
|
78
|
+
"""Report the first path that does not fit the sandbox launch contract.
|
|
79
|
+
|
|
80
|
+
Args:
|
|
81
|
+
worktree_path: the isolated worktree the session runs in.
|
|
82
|
+
settings_path: the minimal safety settings file.
|
|
83
|
+
task_file_path: the file holding the proof-of-concept task text.
|
|
84
|
+
|
|
85
|
+
Returns:
|
|
86
|
+
An error message for the first path that is absent or the wrong
|
|
87
|
+
kind, or None when the worktree is a directory and both files exist.
|
|
88
|
+
"""
|
|
89
|
+
if not worktree_path.is_dir():
|
|
90
|
+
return f"worktree is not a directory: {worktree_path}"
|
|
91
|
+
if not settings_path.is_file():
|
|
92
|
+
return f"settings file not found: {settings_path}"
|
|
93
|
+
if not task_file_path.is_file():
|
|
94
|
+
return f"task file not found: {task_file_path}"
|
|
95
|
+
return None
|
|
96
|
+
|
|
97
|
+
|
|
98
|
+
def _run_via_subprocess(
|
|
99
|
+
all_command_tokens: list[str], working_directory: Path, timeout_seconds: int | None
|
|
100
|
+
) -> int:
|
|
101
|
+
try:
|
|
102
|
+
completed_process = subprocess.run(
|
|
103
|
+
all_command_tokens,
|
|
104
|
+
cwd=working_directory,
|
|
105
|
+
timeout=timeout_seconds,
|
|
106
|
+
check=False,
|
|
107
|
+
)
|
|
108
|
+
except subprocess.TimeoutExpired:
|
|
109
|
+
logger.error(
|
|
110
|
+
"sandbox session exceeded the %s-second wall-clock limit; terminated",
|
|
111
|
+
timeout_seconds,
|
|
112
|
+
)
|
|
113
|
+
return LAUNCH_TIMEOUT_EXIT_CODE
|
|
114
|
+
except OSError as launch_error:
|
|
115
|
+
logger.error(
|
|
116
|
+
"cannot launch the claude executable %s: %s",
|
|
117
|
+
all_command_tokens[0],
|
|
118
|
+
launch_error,
|
|
119
|
+
)
|
|
120
|
+
return LAUNCH_MISSING_PATH_EXIT_CODE
|
|
121
|
+
return completed_process.returncode
|
|
122
|
+
|
|
123
|
+
|
|
124
|
+
def run_sandbox(
|
|
125
|
+
worktree_path: Path,
|
|
126
|
+
settings_path: Path,
|
|
127
|
+
task_text: str,
|
|
128
|
+
timeout_seconds: int | None,
|
|
129
|
+
command_runner: SandboxCommandRunner,
|
|
130
|
+
) -> int:
|
|
131
|
+
"""Run the headless sandbox command in the worktree and return its code.
|
|
132
|
+
|
|
133
|
+
Args:
|
|
134
|
+
worktree_path: the isolated worktree the session runs in.
|
|
135
|
+
settings_path: the minimal safety settings file.
|
|
136
|
+
task_text: the proof-of-concept task the session builds.
|
|
137
|
+
timeout_seconds: the wall-clock limit, or None for no limit.
|
|
138
|
+
command_runner: the callable that runs the command vector.
|
|
139
|
+
|
|
140
|
+
Returns:
|
|
141
|
+
The exit code the command runner reports for the session, which is
|
|
142
|
+
the timeout exit code when the session outran its wall-clock limit.
|
|
143
|
+
"""
|
|
144
|
+
sandbox_command = build_sandbox_command(task_text, settings_path)
|
|
145
|
+
return command_runner(sandbox_command, worktree_path, timeout_seconds)
|
|
146
|
+
|
|
147
|
+
|
|
148
|
+
def _emit_sandbox_summary(
|
|
149
|
+
worktree_path: Path, settings_path: Path, exit_code: int
|
|
150
|
+
) -> None:
|
|
151
|
+
summary = {
|
|
152
|
+
SUMMARY_KEY_WORKTREE: str(worktree_path),
|
|
153
|
+
SUMMARY_KEY_SETTINGS: str(settings_path),
|
|
154
|
+
SUMMARY_KEY_EXIT_CODE: exit_code,
|
|
155
|
+
}
|
|
156
|
+
sys.stdout.write(json.dumps(summary) + "\n")
|
|
157
|
+
|
|
158
|
+
|
|
159
|
+
def _parse_arguments(all_arguments: list[str] | None) -> argparse.Namespace:
|
|
160
|
+
parser = argparse.ArgumentParser(description=__doc__)
|
|
161
|
+
parser.add_argument("--worktree", required=True, help="the sandbox worktree")
|
|
162
|
+
parser.add_argument("--settings", required=True, help="the safety settings file")
|
|
163
|
+
parser.add_argument("--task-file", required=True, help="the task text file")
|
|
164
|
+
parser.add_argument(
|
|
165
|
+
"--timeout-seconds",
|
|
166
|
+
type=int,
|
|
167
|
+
default=DEFAULT_TIMEOUT_SECONDS,
|
|
168
|
+
help="the wall-clock limit for the session",
|
|
169
|
+
)
|
|
170
|
+
return parser.parse_args(all_arguments)
|
|
171
|
+
|
|
172
|
+
|
|
173
|
+
def main(all_arguments: list[str] | None = None) -> int:
|
|
174
|
+
"""Resolve and validate the paths, run the session, and print a summary.
|
|
175
|
+
|
|
176
|
+
Args:
|
|
177
|
+
all_arguments: the command-line arguments, or None to read sys.argv.
|
|
178
|
+
|
|
179
|
+
Returns:
|
|
180
|
+
The sandbox exit code, 2 when a required path is missing, or the
|
|
181
|
+
timeout exit code when the session outran its wall-clock limit.
|
|
182
|
+
"""
|
|
183
|
+
logging.basicConfig(format=LOGGING_FORMAT)
|
|
184
|
+
arguments = _parse_arguments(all_arguments)
|
|
185
|
+
worktree_path = Path(arguments.worktree).expanduser().resolve()
|
|
186
|
+
settings_path = Path(arguments.settings).expanduser().resolve()
|
|
187
|
+
task_file_path = Path(arguments.task_file).expanduser().resolve()
|
|
188
|
+
path_error = validate_sandbox_paths(worktree_path, settings_path, task_file_path)
|
|
189
|
+
if path_error is not None:
|
|
190
|
+
logger.error("%s", path_error)
|
|
191
|
+
return LAUNCH_MISSING_PATH_EXIT_CODE
|
|
192
|
+
task_text = task_file_path.read_text(encoding=TEXT_ENCODING_UTF8)
|
|
193
|
+
exit_code = run_sandbox(
|
|
194
|
+
worktree_path,
|
|
195
|
+
settings_path,
|
|
196
|
+
task_text,
|
|
197
|
+
arguments.timeout_seconds,
|
|
198
|
+
_run_via_subprocess,
|
|
199
|
+
)
|
|
200
|
+
_emit_sandbox_summary(worktree_path, settings_path, exit_code)
|
|
201
|
+
return exit_code
|
|
202
|
+
|
|
203
|
+
|
|
204
|
+
if __name__ == "__main__":
|
|
205
|
+
raise SystemExit(main(sys.argv[1:]))
|