Repository navigation
Conversation
Add the curated release notes, which the release workflow checks and publishes, and drop the statement that @elixir-cloud/tes is private and not released from the release docs. Refs #580 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Reviewer's GuideConfigurationBump the root and released packages to 2.0.0-alpha.51, including TES now that it is public.
DocumentationRead the curated release notes alongside the updated guidance on TES publication and staged-package approval order.
Lock files and vendored codeCheck that the lockfile version metadata matches the release version bump.
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Documentation build overview
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #582 +/- ##
=======================================
Coverage 75.31% 75.31%
=======================================
Files 37 37
Lines 9138 9138
Branches 470 470
=======================================
Hits 6882 6882
Misses 2245 2245
Partials 11 11 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Linked issuesChecked at commit 99a4b01 #580 build: release @elixir-cloud/tes with providers · 3 of 4 met · 1 to check by hand
|
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="packages/ecc-client-ga4gh-tes/package.json" line_range="5" />
<code_context>
{
"name": "elixir-cloud-components",
- "version": "2.0.0-alpha.50",
+ "version": "2.0.0-alpha.51",
"lockfileVersion": 3,
"requires": true,
</code_context>
<issue_to_address>
**TES install fails during partial release**
When the maintainer approves `@elixir-cloud/tes@2.0.0-alpha.51` before `@elixir-cloud/design@2.0.0-alpha.51` is public, npm resolves `@elixir-cloud/tes`'s pinned runtime dependency `@elixir-cloud/design@2.0.0-alpha.51` and fails because that version is not public yet, so users cannot install TES. This also happens if approval stops after TES is published but before design is approved.
Require dependency-first approval and do not approve TES until its pinned internal dependencies are public.
</issue_to_address>Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: packages/ecc-client-ga4gh-tes/package.json:5
Released packages pin the exact versions of the others, so a package approved before its dependencies can't be installed until they are public. Say in which order to approve the staged versions. Refs #580 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Refs #580 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sourcery withdrew this approval because the latest commits introduced blocking findings.
Summary
Version bump and curated notes for
2.0.0-alpha.51, as described in "Releases" in the contributing guide. It is the first release with@elixir-cloud/teson the provider architecture (#443, #516): npm still has1.0.1as itslatest, without providers, and apps need a provider that adds the user's access token to every TES request.node scripts/release.js set 2.0.0-alpha.51sets the version of the root and of all 10 released packages (now including@elixir-cloud/tes, which chore(ecc-utils-ga4gh-tes): tes migration #516 made public), the matching entries inpackage-lock.json, andversionanddate-released(2026-11-14) inCITATION.cff.docs/releases/2.0.0-alpha.51.md: the curated notes, which the release workflow checks inprepareand publishes (first release since ci: align the release workflow #572 added this step).@elixir-cloud/tesis private and not released, and says to approve the staged versions dependencies first (design, thendrs,trs,service-registry, then the rest), from Sourcery's review.What #580 asks for:
devthrough chore(ecc-utils-ga4gh-tes): tes migration #516.run-create.tsbuildsrequest.outputs, andrun-create.test.tschecks that a submitted form sendsoutputs. The notes list it under "Fixes".dev(docs/tes/providers.mdx, "Custom Providers", aRestTesProvidersubclass that overridesfetchand setsAuthorizationon every request; plus the genericTokenWesProviderpattern indocs/providers.mdx).@elixir-cloud.Light lane: version metadata, release notes and one sentence of the release docs; no code. I reviewed the diff myself.
Closes #580
How it was verified
node scripts/release.js check 2.0.0-alpha.51: passes (exit 0).node scripts/release.js dist-tag 2.0.0-alpha.51:latest(2.0.0 prereleases stay onlatestuntil 2.0.0).node scripts/release.js packages: 10 packages, includingpackages/ecc-client-ga4gh-tes.git diffofpackage-lock.json: only the 12versionlines.publish.py check docs/releases/2.0.0-alpha.51.mdofelixir-cloud-aai/.githubat the pinned commit of the workflow (31f22fc, v1.1.0): no findings (a file with aTODOand a bullet without references fails, as expected).node --test scripts/release.test.js scripts/utils.test.js: 27 of 27 pass.npm run lint(includes the build of all packages and the docs): 21 of 21 tasks pass.npmjob:node scripts/release.js pin 2.0.0-alpha.51pins the 11 internal dependencies (reverted afterwards), the build passes, andnpm pack --dry-runof@elixir-cloud/tespacks2.0.0-alpha.51withdist/providers/index.js(exportsRestTesProvider),dist/react/index.js,custom-elements.json,README.mdandLICENSE(67 files).CI passed, Lint, PR title, Conflict markers, Test on Node 22 and 24, CodeQL, Read the Docs).dev: none.Declined review findings
None. Sourcery's finding (approve the staged dependencies first) is about the approval on npm; the contributing guide now says so (804d673).
Open questions
None for this pull request. Steps before the tag are in the handover for the maintainer.
🤖 Generated with Claude Code
Summary by Sourcery
Release version 2.0.0-alpha.51 with the public provider-based TES package, service-registry and accessibility improvements, and curated release documentation.
New Features:
Bug Fixes:
Enhancements:
CI:
Documentation:
Chores: