Skip to content

refactor(orchestrator): replace SDLC actions with protocols - #517

Open
alexsoyes wants to merge 28 commits into
nextfrom
refactor/sdlc-orchestration
Open

refactor(orchestrator): replace SDLC actions with protocols#517
alexsoyes wants to merge 28 commits into
nextfrom
refactor/sdlc-orchestration

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

What

Moves SDLC orchestration from aidd-dev to aidd-orchestrator and replaces the rigid action tree with three routing protocols:

$request → Frame → Deliver → Check → draft PR

  • Recipe skills own validation and retry loops.
  • @executor implements, validates, debugs unclear failures, and commits.
  • A fresh @checker reviews and challenges the outcome independently.
  • Product gaps return to Frame; implementation findings return to Deliver, in parallel through /aidd-dev:10-todo when independent.
  • The flow acts autonomously unless money, an irreversible action, or user authority is involved.

Verification

  • Review: 24/24, no findings
  • 4 Mermaid diagrams rendered
  • 5 flat distributions built
  • 0 broken Markdown links

✅ I certify

  • I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

alexsoyes added 26 commits July 25, 2026 09:28
Keep the orchestrator moving until validation, independent review, and the final outcome challenge all pass. Route contract and implementation gaps back through explicit bounded loops.
Remove the remaining interactive and mandatory-spec descriptions so every public path points to the same autonomous contract.
Record the clean three-axis review after validation loops, fresh checker inputs, bounded Todo fan-out, and public guidance were aligned.
Keep Mermaid ownership links and agent capability declarations explicit so responsibility maps resolve to installed providers.
Put autonomous invariants before each flowchart and let Mermaid remain the single source for transitions instead of repeating every edge in trailing bullets.
Refresh the evidence after canonical agent permissions, the repaired codebase map, and behavior-first protocol documentation.
Use vertical stage subgraphs so each protocol remains readable in a normal Markdown viewport without changing its routing contract.
Keep each protocol self-contained and human-readable while preserving autonomous routing, useful parallelism, and review freshness without exposing Git jargon.
Move validation gates into their owning recipe skills so the orchestration references teach only behavior, delegation, and handoffs.
Record the simplified source handoff, delegated gates, Mermaid renders, and five target builds.
The diagrams describe an already-running orchestration, so routing now starts directly from each zone input.
Record that internal diagrams no longer represent the already-invoked orchestrator as a node.
Keep confirmation boundaries in the SDLC contract and route unclear validation failures through the executor debugging capability.
Record the executor debug fallback and the refined SDLC authority boundary.
@alexsoyes alexsoyes moved this from Todo to Done in AIDD Roadmap Jul 25, 2026
@alexsoyes
alexsoyes marked this pull request as ready for review July 25, 2026 16:46
@alexsoyes

Copy link
Copy Markdown
Contributor Author

Hello @blafourcade ,

J'ai documenté les flows + déplacé l'orchestration dans le bon plugin + viré les actions pour les rendre plus simples / suivables !

Comment thread plugins/aidd-dev/agents/checker.md
Comment thread plugins/aidd-orchestrator/skills/01-sdlc/SKILL.md
@@ -0,0 +1,48 @@
# 01 - Frame

## Behavior

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

On sort completement de la structure des autres skills. Pourquoi pas mais à challenger et ça a pour conséquence : définir un nouveau type de skill a generer côté skill-generate, dans les cours, dans la formation

subgraph SourceStage["Resolve the source"]
direction TB
Source["$source"]
Ticket["/aidd-pm:01-ticket-info"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Trop orienté pour moi on doit être haut niveau ici

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Le but c'est de définir un flot sur lequel l'IA peut se rattacher et suivre une ligne directrice. Qu'est-ce qui ne te convient pas ici ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Les plugins sont optionnels donc ici tu as un risque que ça n'existe pas. Si quelqu'un a une autre façon de faire tu l'oblige a suivre cette façon de faire.

Vu qu'on est sur de l'orchestration, en ayant discuté aussi avec Jerem on est aligné pour utiliser le concept "Ticketing Information" ou autre vs un skill en dur

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Je suis pas sur de comprendre on est sur un skill d'orchestration donc ofc y'a de la dépendance -- tu aurais fait quoi ?

subgraph FrameStage["Frame the contract when needed"]
direction TB
Brainstorm["/aidd-refine:01-brainstorm"]
Spec["/aidd-pm:04-spec"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pareil ici

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

C'est une bonne chose, ça sert de doc à l'IA + aux humains ?

Un dev ça cross les barrières en fonction

Comment thread plugins/aidd-orchestrator/skills/01-sdlc/references/01-frame.md
@blafourcade blafourcade assigned alexsoyes and unassigned blafourcade Jul 27, 2026
@alexsoyes alexsoyes assigned blafourcade and unassigned alexsoyes Jul 28, 2026
@alexsoyes

Copy link
Copy Markdown
Contributor Author

@blafourcade du coup, tu voudrais que je mette ça dans des dossiers de référence ?

Ça ferait plus de sens pour toi sur le flow ?

@blafourcade blafourcade assigned alexsoyes and unassigned blafourcade Jul 28, 2026
@blafourcade

Copy link
Copy Markdown
Contributor

@blafourcade du coup, tu voudrais que je mette ça dans des dossiers de référence ?

Ça ferait plus de sens pour toi sur le flow ?

Oui je pense

@blafourcade blafourcade moved this from Done to In review in AIDD Roadmap Jul 30, 2026

@alexsoyes alexsoyes left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@blafourcade sounds good ?

@alexsoyes
alexsoyes enabled auto-merge (squash) July 31, 2026 05:29
Keep the current PM catalog additions while preserving the relocated synchronous SDLC documentation. Follow next's intentional removal of the frozen v3-to-v4 upgrade guide.
@alexsoyes
alexsoyes requested a review from a team as a code owner July 31, 2026 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants