Skip to content

chore(deps): bump ultramodern packages to 3.9.0-ultramodern.26 - #1034

Merged
BleedingDev merged 2 commits into
mainfrom
chore/ultramodern-26
Sep 30, 2026
Merged

BleedingDev merged 2 commits into
mainfrom
chore/ultramodern-26

Conversation

@BleedingDev

@BleedingDev BleedingDev commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Why the change

The Cloudflare stage deploy fails cloudflare:proof because the ultramodern.25 proof checks for the static topology marker, but deploy builds bake a release marker; ultramodern.26 fixes that check.

Special things to note

  • Mostly a version bump: pnpm catalog, the minimumReleaseAgeExclude list and the lockfile.
  • The API-only proof test mocked the old static marker; it now serves the release identity from the same resolver the proof uses, so it passes on a clean CI checkout.
  • The proof in .26 now reads the expected marker from @modern-js/app-tools-extensions/release-identity, so the marker it expects matches what the build baked.

Change outline

 app/pnpm-workspace.yaml
   catalog.ultramodern:
-    '@modern-js/*': 'npm:@bleedingdev/modern-js-*@3.9.0-ultramodern.25'
+    '@modern-js/*': 'npm:@bleedingdev/modern-js-*@3.9.0-ultramodern.26'
   minimumReleaseAgeExclude:
-    - '@bleedingdev/modern-js-*@3.9.0-ultramodern.25'
+    - '@bleedingdev/modern-js-*@3.9.0-ultramodern.26'
 app/pnpm-lock.yaml   (regenerated)
 app/scripts/tests/api-only-tooling.test.mts
-  mock marker = build.deliveryUnit.buildMarker
+  mock marker = resolveUltramodernReleaseIdentity({ unitId, generationBuildMarker, workspaceRoot }).buildMarker
 proof-cloudflare-version.mjs (from the create package)
-  expected = topology.deliveryUnit.buildMarker
+  expected = resolveUltramodernReleaseIdentity(unit, genMarker, revision).buildMarker

@semanticdiff-com

semanticdiff-com Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Review changes with  SemanticDiff

Changed Files
File Status
  app/pnpm-lock.yaml  70% smaller
  app/pnpm-workspace.yaml  1% smaller
  app/scripts/tests/api-only-tooling.test.mts  0% smaller

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 44 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 7f84075e-02b3-4773-a81f-dc4651290584

📥 Commits

Reviewing files that changed from the base of the PR and between 9b5e95a and 91b5f8e.

⛔ Files ignored due to path filters (1)
  • app/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • app/pnpm-workspace.yaml
  • app/scripts/tests/api-only-tooling.test.mts
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@BleedingDev
BleedingDev merged commit 5da630c into main Sep 30, 2026
21 checks passed
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.

1 participant