Skip to content

feat(skills): route whole-process builds in pipefy-building - #851

Open
adriannoes wants to merge 1 commit into
ad-dev/feat/process-design-specificationfrom
ad-dev/feat/building-router-whole-process
Open

adriannoes wants to merge 1 commit into
ad-dev/feat/process-design-specificationfrom
ad-dev/feat/building-router-whole-process

Conversation

@adriannoes

@adriannoes adriannoes commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

This PR stacks on #850. After #850 merges, retarget it to dev, then close and reopen it to start full CI.

Partial contribution to #780: the router asks before it reuses an existing part.

Summary

  • One part goes to its domain skill. A whole process with an open business decision goes to pipefy-process-design first.
  • A whole process with no blocking decision goes to an available catalog builder or the domain skills. A user-supplied build method takes precedence.
  • The router names four authorization levels and the five off cases defined in feat(skills): return a process specification from pipefy-process-design #850.
  • Selection comes before AI consent. An explicit AI request or an accepted AI line supplies consent. Reuse needs separate consent.
  • A dependency that supplies a required ID comes first. A connector field therefore follows its table.

Accepted limits

Until #852 merges, the automation and agent skills retain their earlier consent wording and can ask about AI again.

The routing test requires a row for every other catalog skill and rejects unknown skill names. It also limits the router's length.

Test plan

  • Clean combined stack: 75 focused tests passed. Linters passed for 19 skills and 57 references.
  • Plugin packaging and Claude validation passed. Claude loaded the catalog, and the skills CLI listed all 19 skills.
  • Merge simulations against current dev and docs(skills): align AI selection, consent, and agent start state with pipefy-process-design #852: no conflict.
  • DCO and skills checks for the latest commit passed.
  • Full CI after retargeting to dev.
  • Cursor-specific smoke: not run.
  • Autonomous run of the final instructions: not run. The new consumer assessment was offline.

Docs / skills

  • docs/parity.md updated when MCP ↔ CLI coverage changed (no coverage change)
  • Affected skills in plugins/pipefy/skills/ updated in this PR (or a paired PR)
  • No docs/skills update needed

Legal / contributions

  • Commits include DCO sign-off (git commit -s)
  • Regulated-domain skills include COMPLIANCE.md when applicable (none added)

@adriannoes adriannoes added documentation Docs, skills, README, or in-repo guides only (no runtime change) enhancement New capability or intentional behavior change (not a bugfix) labels Nov 5, 2026
@adriannoes adriannoes self-assigned this Nov 5, 2026
@adriannoes
adriannoes marked this pull request as ready for review November 5, 2026 16:12
@adriannoes
adriannoes marked this pull request as draft November 6, 2026 00:53
@adriannoes
adriannoes force-pushed the ad-dev/feat/building-router-whole-process branch from 4aade40 to 3cc3d83 Compare November 6, 2026 16:36
@adriannoes
adriannoes marked this pull request as ready for review November 6, 2026 16:38
Signed-off-by: Adrianno Esnarriaga <hey@adrianno.es>
@adriannoes
adriannoes force-pushed the ad-dev/feat/process-design-specification branch from 5d06fa1 to b79ef7e Compare November 9, 2026 13:19
@adriannoes
adriannoes force-pushed the ad-dev/feat/building-router-whole-process branch from 42af4d9 to d320984 Compare November 9, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Docs, skills, README, or in-repo guides only (no runtime change) enhancement New capability or intentional behavior change (not a bugfix)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant