[DIC-28] crux-garden --write: persist gardening report to docs/status/generated/gardening_reports/#8518
Draft
an0mium wants to merge 2 commits into
Draft
[DIC-28] crux-garden --write: persist gardening report to docs/status/generated/gardening_reports/#8518an0mium wants to merge 2 commits into
an0mium wants to merge 2 commits into
Conversation
…/generated/gardening_reports/ Adds --write and --output-dir flags to the `aragora crux-garden` CLI so the GardeningReport is persisted as a timestamped JSON file, satisfying the acceptance criterion in issue #6222 that was unmet by the stdout-only implementation. Mirrors the DIC-26 --write slice pattern (DIC-26 PR #8288 explicitly deferred DIC-28 persistence as a separate slice). Flag-gated: ARAGORA_CRUX_GARDENING_ENABLED (unchanged); --write must be passed explicitly. Live queue effect: none. Advances issue #6222 (DIC-28 Proactive Crux Gardening).
Fixes lint-run CI failure on PR #8518 — ruff format --check flagged the test file as needing canonical reformatting. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017c3yxV3z2oDgfrV2mXhxae
This was referenced Jun 19, 2026
Collaborator
|
Queue-drain close: closing this stale ownerless vision-incubator draft to reduce backlog pressure. Live checks before closing:
This does not delete the branch or discard the commits; the branch is preserved for deliberate revival if the crux-garden write mode should continue. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Slice
Adds
--writeand--output-dirflags to thearagora crux-gardenCLI so theGardeningReportis persisted as a timestamped JSON file underdocs/status/generated/gardening_reports/(or a caller-supplied directory), satisfying the acceptance criterion in issue #6222 that was unmet by the stdout-only implementation. Mirrors the DIC-26--writeslice pattern (DIC-26 PR #8288 explicitly deferred DIC-28 persistence as a separate slice in its "Out of scope" section).Gating
ARAGORA_CRUX_GARDENING_ENABLED=1required;--writemust be passed explicitly)Tests
New in
tests/cli/test_dic28_crux_garden.py(4 new; 14 pre-existing = 18 passed total):test_write_flag_creates_report_file--writeproduces exactly onegardening_report_*.jsonin the output dirtest_write_flag_report_json_is_validgenerated_at,resolved_results,outstanding_results,schema_versiontest_write_flag_creates_output_dir_if_absent--writecreates the full directory tree when absent (3-level nested path)test_write_flag_off_leaves_no_file--write, no report file is created; output dir untouchedAlso adds a
yamlstub at the top of the test file (same pattern as DIC-21/23/25) so tests run cleanly in the hermetic uv-tool pytest venv where pyyaml is absent.Validation
pytest tests/cli/test_dic28_crux_garden.py --noconftest— 18 passed in 0.16 sruff check aragora/cli/commands/dic28_crux_garden.py tests/cli/test_dic28_crux_garden.py— cleanmypy aragora/cli/commands/dic28_crux_garden.py --ignore-missing-imports— Success: no issues foundOut of scope
docs/status/generated/gardening_reports/path on CI (operator trigger decision, not a code gate)--writeoutput path (tracked separately)GardeningConfig.fragility_shift_thresholdhook exists; wiring it to a live DIC-25 catalog requires the Foreman gate to open)Gate status
Per
docs/status/NEXT_STEPS_CANONICAL.md,DIC-13..22(and by extension DIC-23..28) are in the Delay track. The proof-first Foreman gate has not opened. This PR is planning truth and flag-gated operator tooling only; noboss-readylabel applied. Kept as draft.Generated by Claude Code