Skip to content

Support branded dependency IDs - #3095

Open
XiNiHa wants to merge 3 commits into
graphile:mainfrom
XiNiHa:branded-dep-id
Open

Support branded dependency IDs#3095
XiNiHa wants to merge 3 commits into
graphile:mainfrom
XiNiHa:branded-dep-id

Conversation

@XiNiHa

@XiNiHa XiNiHa commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Description

This PR updates Step.addDependency() to return a typed dependency ID instead of a plain number, which can then be used with methods like Step.getDep() and ExecutionValues.at() to retrieve the typed step/value.

Performance impact

These are type-level changes.

Security impact

No changes to runtime behavior.

Checklist

  • My code matches the project's code style and yarn lint:fix passes.
  • I've added tests for the new feature, and yarn test passes.
  • I have detailed the new feature in the relevant documentation.
  • I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
  • If this is a breaking change I've explained why.
@changeset-bot

changeset-bot Bot commented Jul 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 1bdc4f8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
grafast Minor
postgraphile Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@XiNiHa
XiNiHa marked this pull request as ready for review July 11, 2026 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant