Skip to content

docs: add RFC for enhanced dependsOn syntax#322

Draft
branchseer wants to merge 3 commits intomainfrom
depends-on-rfc
Draft

docs: add RFC for enhanced dependsOn syntax#322
branchseer wants to merge 3 commits intomainfrom
depends-on-rfc

Conversation

@branchseer
Copy link
Copy Markdown
Member

Summary

  • Proposes two styles for enhanced dependsOn entries that reuse the existing vp run CLI mental model instead of inventing new symbols
  • Style 1 (CLI string): entries written as CLI args, e.g. "--transitive build", ["--filter", "@myorg/core", "build"]
  • Style 2 (object): keys mirror CLI flag names, e.g. { "transitive": true, "task": "build" }

Test plan

  • Review RFC content for correctness and completeness

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@branchseer branchseer marked this pull request as draft April 6, 2026 17:28
branchseer and others added 2 commits April 7, 2026 09:14
Task name is passed as the value of the flag key (e.g. `{ "transitive": "build" }`)
instead of a separate `task` field with a boolean flag.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@fengmk2
Copy link
Copy Markdown
Member

fengmk2 commented Apr 7, 2026

+1 for "Object syntax"

@ubugeeei
Copy link
Copy Markdown
Collaborator

ubugeeei commented Apr 7, 2026

btw, what happens in this case when there’s a circular reference?

@kazupon
Copy link
Copy Markdown
Contributor

kazupon commented Apr 8, 2026

btw, what happens in this case when there’s a circular reference?

I think Error::CycleDependencyDetected will be occured.

@branchseer
Copy link
Copy Markdown
Member Author

btw, what happens in this case when there’s a circular reference?

Yes Error::CycleDependencyDetected:

Error: Cycle dependency detected: error-cycle-dependency-test#task-a -> error-cycle-dependency-test#task-b -> error-cycle-dependency-test#task-a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants