Skip to content

Commit 494b5cd

Browse files
JoeOakhartNavaJoeOakhartNava
committed
Merge branch 'worktree-20260319-170640'
2 parents eab52de + 8a68e0c commit 494b5cd

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

56 files changed

+6259
-826
lines changed

‎.tickets/.index.json‎

Lines changed: 216 additions & 750 deletions
Large diffs are not rendered by default.

‎.tickets/dso-2cy8.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
id: dso-2cy8
3-
status: in_progress
3+
status: closed
44
deps: []
55
links: []
66
created: 2026-03-19T23:40:03Z

‎.tickets/dso-48fu.md‎

Lines changed: 58 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
id: dso-48fu
3-
status: open
3+
status: closed
44
deps:
55
- dso-gfl9
66
- dso-6576
@@ -42,3 +42,60 @@ parent: dso-2cy8
4242
## Escalation Policy
4343

4444
**Escalation policy**: Escalate to the user whenever you do not have high confidence in your understanding of the work, approach, or intent. "High confidence" means clear evidence from the codebase or ticket context — not inference or reasonable assumption. When in doubt, stop and ask rather than guess.
45+
46+
## Notes
47+
48+
**2026-03-20T00:55:23Z**
49+
50+
COMPLEXITY_CLASSIFICATION: COMPLEX
51+
52+
## ACCEPTANCE CRITERIA
53+
54+
- [ ] Dryrun preview presents a flat list of outcomes without script/skill distinction
55+
Verify: grep -q "flat.*list\|outcome.*list\|planned.*actions" plugins/dso/skills/project-setup/SKILL.md
56+
- [ ] Setup conclusion displays manual steps and environment exports
57+
Verify: grep -q "manual.*steps\|environment.*export\|still need" plugins/dso/skills/project-setup/SKILL.md
58+
- [ ] Tests verify dryrun preview format and conclusion content
59+
Verify: test -f tests/skills/test-project-setup-dryrun-conclusion.sh || test -f tests/skills/test_project_setup_dryrun.py
60+
61+
<!-- note-id: e0aca1xj -->
62+
<!-- timestamp: 2026-03-20T01:18:47Z -->
63+
<!-- origin: agent -->
64+
<!-- sync: unsynced -->
65+
66+
CHECKPOINT 1/6: Task context loaded ✓
67+
68+
<!-- note-id: io6pkfcg -->
69+
<!-- timestamp: 2026-03-20T01:19:09Z -->
70+
<!-- origin: agent -->
71+
<!-- sync: unsynced -->
72+
73+
CHECKPOINT 2/6: Code patterns understood ✓
74+
75+
<!-- note-id: 8gmc1zdv -->
76+
<!-- timestamp: 2026-03-20T01:20:06Z -->
77+
<!-- origin: agent -->
78+
<!-- sync: unsynced -->
79+
80+
CHECKPOINT 3/6: Tests written ✓
81+
82+
<!-- note-id: q0l0tqu2 -->
83+
<!-- timestamp: 2026-03-20T01:21:26Z -->
84+
<!-- origin: agent -->
85+
<!-- sync: unsynced -->
86+
87+
CHECKPOINT 4/6: Implementation complete ✓
88+
89+
<!-- note-id: nw7aerpg -->
90+
<!-- timestamp: 2026-03-20T01:21:58Z -->
91+
<!-- origin: agent -->
92+
<!-- sync: unsynced -->
93+
94+
CHECKPOINT 5/6: Validation passed ✓
95+
96+
<!-- note-id: mmllkm7a -->
97+
<!-- timestamp: 2026-03-20T01:22:28Z -->
98+
<!-- origin: agent -->
99+
<!-- sync: unsynced -->
100+
101+
CHECKPOINT 6/6: Done ✓

‎.tickets/dso-6576.md‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
id: dso-6576
3-
status: open
3+
status: closed
44
deps:
55
- dso-r2es
66
links: []
@@ -39,3 +39,9 @@ parent: dso-2cy8
3939
## Escalation Policy
4040

4141
**Escalation policy**: Escalate to the user whenever you do not have high confidence in your understanding of the work, approach, or intent. "High confidence" means clear evidence from the codebase or ticket context — not inference or reasonable assumption. When in doubt, stop and ask rather than guess.
42+
43+
## Notes
44+
45+
**2026-03-20T00:37:02Z**
46+
47+
COMPLEXITY_CLASSIFICATION: COMPLEX

‎.tickets/dso-6dp5.md‎

Lines changed: 63 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
id: dso-6dp5
3-
status: open
3+
status: closed
44
deps:
55
- dso-r2es
66
links: []
@@ -36,3 +36,65 @@ parent: dso-2cy8
3636
## Escalation Policy
3737

3838
**Escalation policy**: Escalate to the user whenever you do not have high confidence in your understanding of the work, approach, or intent. "High confidence" means clear evidence from the codebase or ticket context — not inference or reasonable assumption. When in doubt, stop and ask rather than guess.
39+
40+
## ACCEPTANCE CRITERIA
41+
42+
- [ ] Each dependency has an individual AskUserQuestion prompt in SKILL.md
43+
Verify: grep -c "AskUserQuestion.*dependency\|AskUserQuestion.*install\|AskUserQuestion.*acli\|AskUserQuestion.*PyYAML\|AskUserQuestion.*pre-commit" plugins/dso/skills/project-setup/SKILL.md | awk '{exit ($1 < 2)}'
44+
- [ ] Already-installed dependencies are skipped (detection-based)
45+
Verify: grep -q "already.*installed\|skip.*installed\|detected.*installed" plugins/dso/skills/project-setup/SKILL.md
46+
- [ ] Each prompt explains what functionality is unavailable without the dependency
47+
Verify: grep -c "without.*functionality\|required for\|enables\|provides" plugins/dso/skills/project-setup/SKILL.md | awk '{exit ($1 < 2)}'
48+
- [ ] Tests verify per-dependency prompt flow
49+
Verify: test -f tests/skills/test-project-setup-dependencies.sh || test -f tests/skills/test_project_setup_deps.py
50+
51+
## Notes
52+
53+
<!-- note-id: wjha60y0 -->
54+
<!-- timestamp: 2026-03-20T01:19:03Z -->
55+
<!-- origin: agent -->
56+
<!-- sync: unsynced -->
57+
58+
CHECKPOINT 1/6: Task context loaded ✓
59+
60+
<!-- note-id: 5g7s86u9 -->
61+
<!-- timestamp: 2026-03-20T01:19:42Z -->
62+
<!-- origin: agent -->
63+
<!-- sync: unsynced -->
64+
65+
CHECKPOINT 2/6: Code patterns understood ✓ — SKILL.md optional deps section at lines 204-212, test pattern from test-project-setup-commands-format.sh, assert.sh library available
66+
67+
<!-- note-id: fcgcuhtp -->
68+
<!-- timestamp: 2026-03-20T01:20:36Z -->
69+
<!-- origin: agent -->
70+
<!-- sync: unsynced -->
71+
72+
CHECKPOINT 3/6: Tests written ✓ — 10 tests in tests/skills/test-project-setup-dependencies.sh, 7 failing (RED) as expected
73+
74+
<!-- note-id: 3jg9lxq6 -->
75+
<!-- timestamp: 2026-03-20T01:20:45Z -->
76+
<!-- origin: agent -->
77+
<!-- sync: unsynced -->
78+
79+
CHECKPOINT 3/6: Tests written ✓
80+
81+
<!-- note-id: m9y7w133 -->
82+
<!-- timestamp: 2026-03-20T01:23:43Z -->
83+
<!-- origin: agent -->
84+
<!-- sync: unsynced -->
85+
86+
CHECKPOINT 4/6: Implementation complete ✓ — replaced bundled optional deps question with 3 individual AskUserQuestion prompts (acli, PyYAML, pre-commit) each with functionality explanation and skip-if-installed logic
87+
88+
<!-- note-id: 3rf9610u -->
89+
<!-- timestamp: 2026-03-20T01:23:56Z -->
90+
<!-- origin: agent -->
91+
<!-- sync: unsynced -->
92+
93+
CHECKPOINT 5/6: Validation passed ✓ — 11/11 dep tests pass, 13/13 commands-format tests pass, all 4 AC patterns verified
94+
95+
<!-- note-id: 1rynbduk -->
96+
<!-- timestamp: 2026-03-20T01:24:04Z -->
97+
<!-- origin: agent -->
98+
<!-- sync: unsynced -->
99+
100+
CHECKPOINT 6/6: Done ✓ — AC1 PASS (3 individual AskUserQuestion prompts), AC2 PASS (skip-if-installed), AC3 PASS (4 functionality explanations), AC4 PASS (test file exists)

‎.tickets/dso-bwtp.md‎

Lines changed: 85 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,85 @@
1+
---
2+
id: dso-bwtp
3+
status: closed
4+
deps: [dso-ozsx]
5+
links: []
6+
created: 2026-03-20T00:42:42Z
7+
type: task
8+
priority: 1
9+
assignee: Joe Oakhart
10+
parent: dso-6576
11+
---
12+
# IMPL: Update project-setup SKILL.md to include CI auto-detection wizard step
13+
14+
Update plugins/dso/skills/project-setup/SKILL.md Step 3 (Interactive Configuration Wizard) to add a CI configuration sub-section that reads detection output from dso-r2es (project-detect.sh) and presents ci.* keys for user confirmation.
15+
16+
TDD EXEMPTION: This task modifies only static agent instruction content (SKILL.md markdown). No executable conditional logic is added. Criterion 3: 'modifies only static assets (schema migrations with no branching logic, Markdown documentation, static config files) where no executable assertion is possible.'
17+
18+
IMPLEMENTATION STEPS:
19+
1. In plugins/dso/skills/project-setup/SKILL.md, add a CI configuration sub-section after the Jira integration sub-section in Step 3
20+
2. The sub-section should:
21+
a. Check if project-detect.sh output is available (from dso-r2es detection context)
22+
b. If CI workflows detected: present detected ci.workflow_name value for confirmation
23+
c. Prompt for: ci.workflow_name, ci.fast_gate_job, ci.fast_fail_job, ci.test_ceil_job, ci.integration_workflow
24+
d. For each, show auto-detected value (from project-detect.sh output) or indicate 'not detected'
25+
e. Note: 'merge.ci_workflow_name is deprecated — if detected in existing config, migrate to ci.workflow_name'
26+
f. If existing config has merge.ci_workflow_name: show deprecation notice and suggest setting ci.workflow_name
27+
3. Authoritative key descriptions come from docs/CONFIGURATION-REFERENCE.md
28+
29+
FILE: plugins/dso/skills/project-setup/SKILL.md (edit — add CI sub-section to Step 3)
30+
31+
32+
## ACCEPTANCE CRITERIA
33+
34+
- [ ] CI configuration sub-section exists in project-setup SKILL.md Step 3
35+
Verify: grep -q 'ci\.workflow_name\|CI workflow' $(git rev-parse --show-toplevel)/plugins/dso/skills/project-setup/SKILL.md
36+
- [ ] SKILL.md mentions auto-detection of CI workflow from .github/workflows/
37+
Verify: grep -q '\.github/workflows\|project-detect' $(git rev-parse --show-toplevel)/plugins/dso/skills/project-setup/SKILL.md
38+
- [ ] SKILL.md mentions deprecation of merge.ci_workflow_name
39+
Verify: grep -q 'merge\.ci_workflow_name.*deprecated\|deprecated.*merge\.ci_workflow_name' $(git rev-parse --show-toplevel)/plugins/dso/skills/project-setup/SKILL.md
40+
- [ ] No qualified skill references broken (check-skill-refs.sh passes)
41+
Verify: bash $(git rev-parse --show-toplevel)/plugins/dso/scripts/check-skill-refs.sh
42+
43+
## Notes
44+
45+
<!-- note-id: vaye1ftv -->
46+
<!-- timestamp: 2026-03-20T02:00:15Z -->
47+
<!-- origin: agent -->
48+
<!-- sync: unsynced -->
49+
50+
CHECKPOINT 1/6: Task context loaded ✓
51+
52+
<!-- note-id: y5t3yeda -->
53+
<!-- timestamp: 2026-03-20T02:00:19Z -->
54+
<!-- origin: agent -->
55+
<!-- sync: unsynced -->
56+
57+
CHECKPOINT 2/6: Code patterns understood ✓
58+
59+
<!-- note-id: zf5dbme8 -->
60+
<!-- timestamp: 2026-03-20T02:01:08Z -->
61+
<!-- origin: agent -->
62+
<!-- sync: unsynced -->
63+
64+
CHECKPOINT 3/6: Tests written ✓ — 7 RED tests in tests/skills/test_project_setup_ci_autodetection.py
65+
66+
<!-- note-id: zzb5xmtq -->
67+
<!-- timestamp: 2026-03-20T02:01:53Z -->
68+
<!-- origin: agent -->
69+
<!-- sync: unsynced -->
70+
71+
CHECKPOINT 4/6: Implementation complete ✓ — CI auto-detection sub-section added to SKILL.md Step 3 (after Jira section)
72+
73+
<!-- note-id: sjijirue -->
74+
<!-- timestamp: 2026-03-20T02:02:13Z -->
75+
<!-- origin: agent -->
76+
<!-- sync: unsynced -->
77+
78+
CHECKPOINT 5/6: Validation passed ✓ — all 4 AC grep checks pass, check-skill-refs passes, 7 new tests GREEN, 4 pre-existing RED tests unchanged
79+
80+
<!-- note-id: la1p0o5a -->
81+
<!-- timestamp: 2026-03-20T02:02:21Z -->
82+
<!-- origin: agent -->
83+
<!-- sync: unsynced -->
84+
85+
CHECKPOINT 6/6: Done ✓ — All AC items verified. Files changed: plugins/dso/skills/project-setup/SKILL.md (CI sub-section added to Step 3), tests/skills/test_project_setup_ci_autodetection.py (new)

‎.tickets/dso-bzvu.md‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
id: dso-bzvu
3-
status: open
3+
status: closed
44
deps:
55
- dso-r2es
66
links: []
@@ -40,3 +40,9 @@ parent: dso-2cy8
4040
## Escalation Policy
4141

4242
**Escalation policy**: Escalate to the user whenever you do not have high confidence in your understanding of the work, approach, or intent. "High confidence" means clear evidence from the codebase or ticket context — not inference or reasonable assumption. When in doubt, stop and ask rather than guess.
43+
44+
## Notes
45+
46+
**2026-03-20T00:37:02Z**
47+
48+
COMPLEXITY_CLASSIFICATION: COMPLEX

‎.tickets/dso-cz6p.md‎

Lines changed: 58 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
id: dso-cz6p
3-
status: open
3+
status: closed
44
deps:
55
- dso-gfl9
66
- dso-6576
@@ -35,3 +35,60 @@ parent: dso-2cy8
3535
## Escalation Policy
3636

3737
**Escalation policy**: Escalate to the user whenever you do not have high confidence in your understanding of the work, approach, or intent. "High confidence" means clear evidence from the codebase or ticket context — not inference or reasonable assumption. When in doubt, stop and ask rather than guess.
38+
39+
## Notes
40+
41+
**2026-03-20T00:55:23Z**
42+
43+
COMPLEXITY_CLASSIFICATION: COMPLEX
44+
45+
## ACCEPTANCE CRITERIA
46+
47+
- [ ] CLAUDE.md references consolidated ci.* namespace
48+
Verify: grep -q "ci.workflow_name\|ci\.\*" CLAUDE.md
49+
- [ ] CONFIGURATION-REFERENCE.md documents ci.workflow_name and deprecates merge.ci_workflow_name
50+
Verify: grep -q "ci.workflow_name" plugins/dso/docs/CONFIGURATION-REFERENCE.md && grep -qi "deprecat.*merge.ci_workflow_name" plugins/dso/docs/CONFIGURATION-REFERENCE.md
51+
- [ ] INSTALL.md reflects updated setup instructions
52+
Verify: grep -q "project-setup\|dso:project-setup" plugins/dso/docs/INSTALL.md
53+
54+
<!-- note-id: gcc0h03a -->
55+
<!-- timestamp: 2026-03-20T01:38:04Z -->
56+
<!-- origin: agent -->
57+
<!-- sync: unsynced -->
58+
59+
CHECKPOINT 1/6: Task context loaded ✓
60+
61+
<!-- note-id: s69r8zt2 -->
62+
<!-- timestamp: 2026-03-20T01:39:12Z -->
63+
<!-- origin: agent -->
64+
<!-- sync: unsynced -->
65+
66+
CHECKPOINT 2/6: Code patterns understood ✓ — ci.workflow_name is the new consolidated key (preferred); merge.ci_workflow_name is deprecated with fallback in merge-to-main.sh; project-setup wizard now exists as /dso:project-setup skill
67+
68+
<!-- note-id: tnx8x0ea -->
69+
<!-- timestamp: 2026-03-20T01:39:20Z -->
70+
<!-- origin: agent -->
71+
<!-- sync: unsynced -->
72+
73+
CHECKPOINT 3/6: Tests written (none required — docs only) ✓
74+
75+
<!-- note-id: d09cmxuf -->
76+
<!-- timestamp: 2026-03-20T01:41:08Z -->
77+
<!-- origin: agent -->
78+
<!-- sync: unsynced -->
79+
80+
CHECKPOINT 4/6: Implementation complete ✓ — updated CLAUDE.md (ci.workflow_name ci.* section + project-setup quick ref), CONFIGURATION-REFERENCE.md (ci.workflow_name new entry + merge.ci_workflow_name deprecated), INSTALL.md (project-setup wizard step + ci.workflow_name in key summary)
81+
82+
<!-- note-id: 5q9jj832 -->
83+
<!-- timestamp: 2026-03-20T01:41:20Z -->
84+
<!-- origin: agent -->
85+
<!-- sync: unsynced -->
86+
87+
CHECKPOINT 5/6: Validation passed ✓ — check-skill-refs.sh: no unqualified skill refs found
88+
89+
<!-- note-id: 4zh5ybh0 -->
90+
<!-- timestamp: 2026-03-20T01:41:41Z -->
91+
<!-- origin: agent -->
92+
<!-- sync: unsynced -->
93+
94+
CHECKPOINT 6/6: Done ✓ — all 3 AC verified: (1) CLAUDE.md ci.workflow_name PASS, (2) CONFIGURATION-REFERENCE.md ci.workflow_name + deprecation PASS, (3) INSTALL.md project-setup PASS

0 commit comments

Comments
 (0)