Skip to content

chore: uninstall spectacles SDD suite - #716

Merged
norrietaylor merged 1 commit into
mainfrom
spectacles/uninstall
Jul 10, 2026
Merged

chore: uninstall spectacles SDD suite#716
norrietaylor merged 1 commit into
mainfrom
spectacles/uninstall

Conversation

@norrietaylor

@norrietaylor norrietaylor commented Jul 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Uninstalls the spectacles SDD suite from this repository. This reverses the file artifacts placed by scripts/quick-setup.sh --suite sdd.

Removed in this PR (git tree)

  • 17 workflow wrappers in .github/workflows/: distillery-sync, sdd-spec, sdd-triage, sdd-dispatch, sdd-execute-haiku/-sonnet/-opus, sdd-validate, sdd-review, sdd-derive, sdd-pr-sanitize, sdd-triage-dedupe-tasks, sdd-triage-promote-ready, sdd-monitor, sdd-spike-actuator, sdd-spike-reentry, sdd-status.
  • 4 issue templates — .github/ISSUE_TEMPLATE/{bug,chore,feature,spec}.md (byte-identical to the spectacles source). The repo's own bug_report.md and feature_request.md are kept.
  • docs/spikes/README.md — the seeded spike-author guide. The two real spike outputs (2026-06-20-wireguard-implementation.md, 2026-06-21-gvproxy-attachment.md) are kept.
  • .gitignore — the .serena/ Serena-state block is removed; every other entry is untouched.

Summary by CodeRabbit

  • Chores
    • Removed predefined issue templates for bugs, features, chores, and specifications.
    • Removed automated workflows for specification, triage, validation, review, execution, status updates, spike handling, and synchronization.
    • Updated ignored build-output paths.
    • Removed spike authoring and documentation guidance.

@norrietaylor
chore: uninstall spectacles SDD suite
46ec855 
Remove the file artifacts installed by scripts/quick-setup.sh (spectacles):
the 17 sdd-*/distillery-sync workflow wrappers, the bug/chore/feature/spec
issue templates, the seeded docs/spikes/README.md, and the .serena/ .gitignore
entry. The two real spike outputs under docs/spikes/ are retained.

Settings-level artifacts (sdd:*/model:* labels, DISTILLERY_*/SERENA_/SDD_*
variables, CLAUDE_CODE_OAUTH_TOKEN secret) are not in the git tree and must be
removed separately with gh - see the PR body.
@coderabbitai

coderabbitai Bot commented Jul 10, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The pull request removes the repository’s SDD issue templates, GitHub Actions workflows, spike documentation, and distillery synchronization workflow. It also replaces the .serena/ ignore rule with target/.

Changes

SDD pipeline removal

Layer / File(s) Summary
Remove issue templates
.github/ISSUE_TEMPLATE/*		 Deletes the Bug, Chore, Feature, and Specification templates and their documented SDD command vocabulary.
Remove SDD lifecycle workflows
.github/workflows/sdd-*.yml		 Deletes specification, derivation, triage, dispatch, execution, validation, review, status, monitoring, spike, and deduplication automation.
Remove synchronization and spike documentation
.github/workflows/distillery-sync.yml, docs/spikes/README.md		 Deletes the distillery synchronization workflow and spike authoring protocol.

Repository cleanup

Layer / File(s) Summary
Update ignored paths
.gitignore		 Replaces the .serena/ ignore entry with target/.

Estimated code review effort: 4 (Complex) | ~45 minutes

Possibly related issues

  • gominimal/inbox#70 — Directly concerns removal of the Spectacles SDD workflows and issue templates.
  • gominimal/minimal#616 — Discusses the same SDD pipeline components removed here.

Possibly related PRs

Suggested reviewers: twitchyliquid64, bryan-minimal, evanspearman

Poem

A rabbit hops where workflows ran,
No more SDD trails across the plan.
Templates fade beneath moonlight,
target/ joins the ignore night.
“Clean burrows!” cheers the hare.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: removing the Spectacles SDD suite artifacts.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands.

bryan-minimal
bryan-minimal approved these changes Jul 10, 2026
View reviewed changes

@bryan-minimal bryan-minimal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@norrietaylor
norrietaylor enabled auto-merge (squash) July 10, 2026 22:14
@norrietaylor
norrietaylor merged commit 0281323 into main Jul 10, 2026
27 checks passed
@norrietaylor
norrietaylor deleted the spectacles/uninstall branch July 10, 2026 22:16
This was referenced Jul 23, 2026
Closed
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bryan-minimal bryan-minimal bryan-minimal approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@norrietaylor @bryan-minimal