Enhance SLDD process and SPEC.md support across all skills#2
Open
Enhance SLDD process and SPEC.md support across all skills#2
Conversation
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
…ss for SPEC.md outputs
…ecks across all steps. - Added prerequisite checks for Steps 02 to 06 to ensure prior steps are complete and approved before proceeding. - Implemented skip-ahead detection to prevent premature implementation or testing requests. - Updated SPEC.md resume behavior to include violation checks and context extraction for each step. - Clarified objectives, audience, style, and tone for each step to ensure consistency and clarity. - Improved response structure for draft outputs and save flows, emphasizing user approval before file operations. - Ensured that SPEC.md maintains a strict structure, containing only progress checklists and links to step files.
…nce checks across all steps
…ks across all SLDD skills
…files across all steps
… across multiple SKILL.md files
…ltiple SKILL.md files
omatheusmesmo
requested changes
Apr 21, 2026
Collaborator
omatheusmesmo
left a comment
There was a problem hiding this comment.
@dearrudam , great job, man!
I can see how much effort you put on this, I would love to discuss each skill on a Discord call, but I don't think will be easier rather than do this async,
Let's discuss and define a standard to SLDD-00, then we can discuss the remains.
Simplify approval protocol presentation and trim SLDD wording
Co-authored-by: Copilot <copilot@github.com>
omatheusmesmo
requested changes
May 1, 2026
Collaborator
omatheusmesmo
left a comment
There was a problem hiding this comment.
Review: sldd-88 over-centralization breaks the self-contained model
Core Problem
sldd-88 became a god-skill that every other skill depends on for gates, save decisions, templates, and section headings. This creates two failures:
- Runtime coupling — Skills are loaded one-at-a-time. An agent running sldd-01 does NOT have sldd-88 in context. The instruction "refer to Step 88 Section 6" is a dead link at execution time.
- Semantic loss — Headings without guardrails are structure without substance. "Architecture Diagram" is a heading. "Do not generate implementation code or tests" is a guardrail. They are not interchangeable.
Anti-Patterns
| Anti-Pattern | Where | Why It's a Problem |
|---|---|---|
| Guardrails replaced by headings | Steps 01–06, 99 | Section headings describe what to write, not how to behave. "Do not generate code" is gone from Step 02. Edge-case requirement is gone from Step 01. Version policy specifics are gone from Step 03. |
| Cross-skill reference as instruction | Every skill: "Use sldd-88 for shared gates, save decision, and templates" | If sldd-88 is not loaded, the skill is non-functional. Self-contained > DRY. |
| Independence Rule that contradicts reality | sldd-88 Sec 0: "skills are self-contained" | Every skill depends on sldd-88. The rule is false as written. |
| Two-Phase Verification scope too broad | sldd-88 Sec 4: "every technical step" | Steps 01–03 and 06 produce markdown specs — they don't "modify the codebase." Over-engineering specification steps with action plans. |
| Gate rules in two places | sldd-88 Sec 3 + each skill's "Gate + Resume Checks" | Duplicated = drift risk. Edit one, forget the other. |
Proposal
Delete sldd-88 and sldd-98. They create coupling that costs more than the duplication they save.
| Content from sldd-88/98 | Should live in | Reason |
|---|---|---|
| SPEC.md journal template | sldd-00 | It's the orchestrator — it creates the SPEC.md |
| Artifact directory structure | sldd-00 | Same reason |
| Gate order | sldd-00 | Already has this |
| Behavioral guardrails | Each skill | "Do not generate code", "edge case per criterion", version policy specifics — these guide agent behavior, not file structure |
| Section headings with guidance | Each skill | Headings + per-heading substance, not bare names |
| Gate prerequisites | Each skill | The skill is the authoritative source |
| Save flow | Each skill (1 paragraph) | "Present draft → wait for approval → save artifact → mark [x] in SPEC.md → ask to continue" |
| Two-Phase Protocol | sldd-04 and sldd-05 only | Only implementation steps need action-plan-then-evidence |
| Approval prompts | Each skill (inline) | Standardizing exact prompt wording doesn't justify an 83-line skill dependency |
Result
7 self-contained skills, zero cross-skill dependencies, guardrails back where they matter.
Also Missing
- "Existing codebase" is never defined — add a one-line definition in sldd-00
- No revision protocol — if Step 03 reveals Step 01 was insufficient, there's no way to mark a step as under revision, re-approve, and re-validate downstream
- README.md will be stale — if sldd-88/98 stay, they must appear in the skills table
made with ☕ by @omatheusmesmo and @glm-5.1
…ng spec clarification
Refactor SLDD helpers and enhance SKILL.md clarity
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.
This pull request introduces significant improvements to the SLDD (Spec Loops Driven Development) skills repository, focusing on clarifying the purpose and architecture of executable skills, enhancing documentation, and providing better support for local development. The changes make the SLDD process, agent responsibilities, and installation options more explicit and developer-friendly.
Documentation and Process Clarifications:
AGENTS.mdfile that clearly defines the repository's purpose, global agent behavior, limits, and conventions for maintaining SLDD skills.README.mdto:skills/, with no shared runtime helpers. [1] [2]Development Tooling:
install-sldd-skills.sh, a script for local development that installs SLDD skills as symbolic links, enabling immediate visibility of changes in development environments.Skill Content Consistency:
skills/sldd-00-process-overview-and-navigation-guide/SKILL.mdto:Summary of Most Important Changes:
1. Documentation and Process Architecture
AGENTS.mdto define agent responsibilities, limits, and conventions.README.mdwith a mermaid flowchart, explicit runtime skill architecture, improved installation/dev instructions, and clarified process flows for brownfield and greenfield projects. [1] [2] [3] [4] [5]2. Development Workflow Support
install-sldd-skills.shfor symlink-based local development, allowing immediate reflection of skill edits in development tools.3. Skill File Consistency and Navigation
sldd-00-process-overview-and-navigation-guide/SKILL.mdto enforce step order, artifact handling, resume logic, and approval protocols.These changes collectively improve the maintainability, clarity, and usability of the SLDD skills repository for both users and contributors.