Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

refactor(aidd-orchestrator): make the delivery loop a contract-compliant router

Open
#420 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
48/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Quiet
Tech stack
typescript
Domain
tooling

Research direction

Start with the root router, the routing reference, and the public skill entry point 00-async-dev; trace how route, setup, run, and review currently select and load actions. Compare the shortest and longest paths before and after, preserving provider discovery, trigger aliases, and observable outcomes. Done means the root contains only the contract flow, action table, and transversal rules, with line counts recorded in the PR.

Written by the indexing model from the issue text.

Description

Need

The delivery loop configures unattended delivery, routes one ready work request to delivery, and iterates on review feedback. Its current public name, async-dev, describes an implementation style rather than that outcome.

The public skill remains 00-async-dev in this refactor. Renaming its identifier is a separate compatibility migration; this issue only makes its router understandable, lazy-loaded, and contract-compliant.

Current state

The SDLC provider is already discovered at runtime by capability description. The remaining problem is structural: the always-loaded router duplicates the routing protocol and expands every setup, run, and review detail, although one invocation uses only one path.

Scope

  • Make the root router express only route → setup | run | review, its action table, and transversal rules.
  • Add internal load boundaries:
    • route resolves the mode from the routing protocol.
    • setup, run, and review each load their existing ordered action sequence only after selection.
  • Keep detailed selection precedence, trigger aliases, and conflict handling in the routing reference, with one source of truth.
  • Preserve the existing dynamic SDLC-provider discovery and all current setup, run, and review behavior.
  • Measure the lines read on the shortest and longest paths before and after.

Out of scope

  • Renaming 00-async-dev, changing its invocation, or breaking trigger compatibility.
  • Splitting setup, run, and review into separate public skills.
  • Changing automation policy, GitHub integration, or the SDLC capability contract.

Acceptance criteria

  • The root router satisfies the skill contract: flow, action table, and transversal rules only.
  • A mode is selected through one routing source of truth, then only its internal sequence is loaded.
  • Setup, run, and review produce the same observable outcomes as before for the same inputs.
  • Provider discovery remains runtime-based; no provider name appears in root routing logic.
  • Existing trigger aliases continue to select the same mode.
  • The PR states shortest- and longest-path line counts before and after.
Dominant language
TypeScript
Stars
481
Forks
45
Avg merge
19h 38m
Merged PRs (30d)
108

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ai-driven-dev/framework

All issues in ai-driven-dev/framework

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.