Repository navigation
Add TypeScript SDK capability manifest and compatibility matrix - #71754
Merged
Merged
Conversation
1 task done
jason810496
force-pushed
the
ci/ts-sdk/compat-matrix
branch
from
August 18, 2026 07:28
02aefdf to
f36d5c5
Compare
jason810496
force-pushed
the
ci/ts-sdk/compat-matrix
branch
from
August 26, 2026 02:01
f36d5c5 to
1d2fe9a
Compare
jason810496
marked this pull request as ready for review
August 26, 2026 02:03
jason810496
requested review from
amoghrajesh,
ashb,
bugraoz93,
gopidesupavan,
jscheffl and
potiuk
as code owners
August 26, 2026 02:03
jason810496
force-pushed
the
ci/ts-sdk/compat-matrix
branch
from
August 26, 2026 02:22
1d2fe9a to
7a4253a
Compare
phanikumv
reviewed
Aug 26, 2026
phanikumv
reviewed
Aug 26, 2026
Co-authored-by: Phani Kumar <94376113+phanikumv@users.noreply.github.com>
jason810496
force-pushed
the
ci/ts-sdk/compat-matrix
branch
from
August 26, 2026 04:30
b61b70a to
ebc8f87
Compare
phanikumv
approved these changes
Aug 26, 2026
amoghrajesh
approved these changes
Aug 26, 2026
amoghrajesh
left a comment
amoghrajesh
left a comment
Contributor
There was a problem hiding this comment.
Good work on this one, one issue, otherwise LGTM
read_ts_schema_version returned None both when capabilities.yaml disagreed with the generated runtime constant and when the constant could not be read at all, so a missing or renamed SUPERVISOR_API_VERSION sent the contributor to capabilities.yaml, which cannot fix either case. The unreadable case now names src/generated/supervisor.ts and the generator instead. Reading the file also no longer raises when it is absent, so that path reaches the new message.
jason810496
commented
Aug 26, 2026
imrichardwu
pushed a commit
to imrichardwu/airflow
that referenced
this pull request
Sep 11, 2026
…he#71754) * Add TypeScript SDK capability manifest and compatibility matrix Co-authored-by: Phani Kumar <94376113+phanikumv@users.noreply.github.com> * Report a missing TypeScript supervisor schema constant separately read_ts_schema_version returned None both when capabilities.yaml disagreed with the generated runtime constant and when the constant could not be read at all, so a missing or renamed SUPERVISOR_API_VERSION sent the contributor to capabilities.yaml, which cannot fix either case. The unreadable case now names src/generated/supervisor.ts and the generator instead. Reading the file also no longer raises when it is absent, so that path reaches the new message. --------- Co-authored-by: Phani Kumar <94376113+phanikumv@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The TypeScript SDK needs a machine-readable declaration of its supported Language SDK capabilities.
What
Was generative AI tooling used to co-author this PR?