Research SPDD + Fowler Fragments + cc-connect; promote 2 rules - #7
Merged
Merged
Conversation
Summary: - Logged Thoughtworks SPDD, Fowler's 29 Apr Fragments, and chenhg5/cc-connect as sources. - Promoted spec/code two-way sync and sensor-based verification into AGENTS.md + Copilot mirror. - Added two knowledge notes (SPDD / REASONS Canvas; agent-integration patterns). Changed files: - docs/source-repos.md: 3 rows (SPDD + Fragments promoted; cc-connect extracted). - docs/research-log.md: 3 dated observation blocks, extends/contradicts, promoted-rule rows, pending + anti-pattern entries. - AGENTS.md: spec/code sync bullet (House style) + sensor-verification check (Validation). - .github/copilot-instructions.md: mirrored both rules. - docs/knowledge/structured-prompt-driven-development.md: new descriptive note. - docs/knowledge/agent-integration-patterns.md: new descriptive note. - docs/knowledge/README.md: listed both new notes. - docs/ai-agent-coding-strategy.md: pointer to the SPDD note. Validation: - ./scripts/check.sh: Tier 1 (length, Boundaries) and Tier 2 (markdownlint) clean. - Tier 3/4 binaries absent locally; will verify in CI. - All new files end with a Boundaries section; all under 200 lines. Follow-ups: - Confirm Tier 3 (lychee) and Tier 4 (typos) pass in CI; fix if flagged. - cc-connect left as extracted (knowledge note only, no instruction-file rule). https://claude.ai/code/session_01AmmEaVLpyeJz7TZ3iyXtWK
tomqwu
marked this pull request as ready for review
June 10, 2026 15:26
4 tasks
tomqwu
added a commit
that referenced
this pull request
Jun 10, 2026
Codifies the operating policy this session has been running under: builder agents may merge after the review gate passes (CI green + LGTM marker matching head SHA + no newer blocking feedback), replacing the previous "do not merge unless the user explicitly asks" rule. - AGENTS.md: adds the builder and reviewer prompt snippets, with explicit head-SHA tying. - CLAUDE.md: same builder/reviewer split with a pointer to the knowledge note for scheduler, lock, and state guardrails. - .github/copilot-instructions.md: mirrors the head-SHA approval gate for the Copilot baseline. - docs/knowledge/agentic-pr-review-loop.md: updates the prompts, the validation checklist, and the "let the builder bypass review" framing. Rebased onto current main (post #7 and #8); doc-quality CI passes on the rebased head.
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
2 participants
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.
Summary
Continues the research-handling pipeline for three sources: the Thoughtworks Structured-Prompt-Driven Development (SPDD) article, Martin Fowler's 29 Apr 2026 Fragments, and chenhg5/cc-connect.
docs/source-repos.mdanddocs/research-log.md(SPDD + Fragmentspromoted; cc-connectextracted).AGENTS.mdand its Copilot mirror:docs/knowledge/structured-prompt-driven-development.md— the seven-part REASONS Canvas, the sync flows, fitness assessment.docs/knowledge/agent-integration-patterns.md— bridge-daemon, capability-interface-over-identity-switch, plugin self-registration, versioned preset-manifest patterns from cc-connect; also records cc-connect's duplicated-AGENTS.md/CLAUDE.mdas a cautionary counter-example.cc-connect stayed
extracted(its generic lessons went to a knowledge note + cautionary anti-pattern; no new instruction-file rule).Test plan
doc-quality: Tier 1 (≤200 lines, Boundaries present), Tier 2 (markdownlint), Tier 3 (lychee offline), Tier 4 (typos) all green. Tiers 1–2 verified locally via./scripts/check.sh.## Boundariesand are listed indocs/knowledge/README.md.AGENTS.md↔.github/copilot-instructions.mdmirror stays in sync for the two promoted rules.Follow-ups
openspdd-style per-step CLI and provider-keyed multi-agent config left as pending observations until a runtime exists.https://claude.ai/code/session_01AmmEaVLpyeJz7TZ3iyXtWK
Generated by Claude Code