Skip to content

chore(changeset): remove am-mock-api from changeset and add to ignore list#718

Merged
vatsalparikh merged 1 commit into
mainfrom
remove-am-mock-changeset
Jun 26, 2026
Merged

chore(changeset): remove am-mock-api from changeset and add to ignore list#718
vatsalparikh merged 1 commit into
mainfrom
remove-am-mock-changeset

Conversation

@vatsalparikh

@vatsalparikh vatsalparikh commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes am-mock-api patch entry from the some-shirts-joke.md changeset
  • Adds am-mock-api to the changeset ignore list in config.json so it is excluded from future release tracking

Test plan

  • Verify changeset publish does not attempt to version am-mock-api

Summary by CodeRabbit

  • New Features
    • Added support for PAR in redirect-flow requests.
  • Chores
    • Updated release metadata to include the latest minor versions for several SDK-related packages.
    • Excluded an additional internal package from changeset processing.

@changeset-bot

changeset-bot Bot commented Jun 26, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 387428e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes changesets to release 12 packages
Name Type
@forgerock/sdk-request-middleware Minor
@forgerock/sdk-oidc Minor
@forgerock/davinci-client Minor
@forgerock/oidc-client Minor
@forgerock/journey-client Minor
@forgerock/device-client Minor
@forgerock/protect Minor
@forgerock/sdk-types Minor
@forgerock/sdk-utilities Minor
@forgerock/iframe-manager Minor
@forgerock/sdk-logger Minor
@forgerock/storage Minor

Click here to learn what changesets are, and how to add one.

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

@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c05474df-05e4-4307-b0d5-46679973b693

📥 Commits

Reviewing files that changed from the base of the PR and between bd67404 and 387428e.

📒 Files selected for processing (2)
  • .changeset/config.json
  • .changeset/some-shirts-joke.md
💤 Files with no reviewable changes (1)
  • .changeset/some-shirts-joke.md

📝 Walkthrough

Walkthrough

The PR adds am-mock-api to the Changeset ignore list and rewrites one changeset entry to remove that package, add four minor version bumps, and describe PAR support in oidc-client redirect requests.

Changes

Release metadata

Layer / File(s) Summary
Ignore list and release entry
.changeset/config.json, .changeset/some-shirts-joke.md
The Changeset config now ignores am-mock-api, and the changeset entry replaces the patch entry with four minor bumps plus a PAR support note.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

ᕱᕱ I hop through the release notes,
nibbling am-mock-api from the patchy oats.
Four tiny bumps now dance in a line,
and PAR hops along with redirect time.
🥕

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: moving am-mock-api out of the changeset and into the ignore list.
Description check ✅ Passed The description covers the main changes and test plan, but it does not follow the template's exact headings or include a Jira ticket section.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove-am-mock-changeset

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.

@nx-cloud

nx-cloud Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit 387428e

Command Status Duration Result
nx run-many -t build --no-agents ✅ Succeeded <1s View ↗
nx affected -t build lint test typecheck e2e-ci ✅ Succeeded 2m 19s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-06-26 23:25:16 UTC

@pkg-pr-new

pkg-pr-new Bot commented Jun 26, 2026

Copy link
Copy Markdown

Open in StackBlitz

@forgerock/davinci-client

pnpm add https://pkg.pr.new/@forgerock/davinci-client@718

@forgerock/device-client

pnpm add https://pkg.pr.new/@forgerock/device-client@718

@forgerock/journey-client

pnpm add https://pkg.pr.new/@forgerock/journey-client@718

@forgerock/oidc-client

pnpm add https://pkg.pr.new/@forgerock/oidc-client@718

@forgerock/protect

pnpm add https://pkg.pr.new/@forgerock/protect@718

@forgerock/sdk-types

pnpm add https://pkg.pr.new/@forgerock/sdk-types@718

@forgerock/sdk-utilities

pnpm add https://pkg.pr.new/@forgerock/sdk-utilities@718

@forgerock/iframe-manager

pnpm add https://pkg.pr.new/@forgerock/iframe-manager@718

@forgerock/sdk-logger

pnpm add https://pkg.pr.new/@forgerock/sdk-logger@718

@forgerock/sdk-oidc

pnpm add https://pkg.pr.new/@forgerock/sdk-oidc@718

@forgerock/sdk-request-middleware

pnpm add https://pkg.pr.new/@forgerock/sdk-request-middleware@718

@forgerock/storage

pnpm add https://pkg.pr.new/@forgerock/storage@718

commit: 387428e

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 23.13%. Comparing base (eafe277) to head (387428e).
⚠️ Report is 40 commits behind head on main.

❌ Your project status has failed because the head coverage (23.13%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #718      +/-   ##
==========================================
+ Coverage   18.07%   23.13%   +5.06%     
==========================================
  Files         155      161       +6     
  Lines       24398    25602    +1204     
  Branches     1203     1613     +410     
==========================================
+ Hits         4410     5924    +1514     
+ Misses      19988    19678     -310     

see 16 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

Copy link
Copy Markdown
Contributor

Deployed 3190cc1 to https://ForgeRock.github.io/ping-javascript-sdk/pr-718/3190cc1eeb2adab7f176c0be6876fd9fcb0d2fec branch gh-pages in ForgeRock/ping-javascript-sdk

@github-actions

Copy link
Copy Markdown
Contributor

📦 Bundle Size Analysis

📦 Bundle Size Analysis

🚨 Significant Changes

🔻 @forgerock/device-client - 0.0 KB (-10.0 KB, -100.0%)
🔻 @forgerock/journey-client - 0.0 KB (-92.6 KB, -100.0%)

📊 Minor Changes

📉 @forgerock/device-client - 10.0 KB (-0.0 KB)

➖ No Changes

@forgerock/oidc-client - 35.3 KB
@forgerock/sdk-types - 9.1 KB
@forgerock/davinci-client - 54.4 KB
@forgerock/sdk-utilities - 18.6 KB
@forgerock/journey-client - 92.6 KB
@forgerock/protect - 144.6 KB
@forgerock/iframe-manager - 3.2 KB
@forgerock/sdk-logger - 1.6 KB
@forgerock/storage - 1.5 KB
@forgerock/sdk-request-middleware - 4.6 KB
@forgerock/sdk-oidc - 5.7 KB


14 packages analyzed • Baseline from latest main build

Legend

🆕 New package
🔺 Size increased
🔻 Size decreased
➖ No change

ℹ️ How bundle sizes are calculated
  • Current Size: Total gzipped size of all files in the package's dist directory
  • Baseline: Comparison against the latest build from the main branch
  • Files included: All build outputs except source maps and TypeScript build cache
  • Exclusions: .map, .tsbuildinfo, and .d.ts.map files

🔄 Updated automatically on each push to this PR

@vatsalparikh vatsalparikh merged commit 295657e into main Jun 26, 2026
9 checks passed
@vatsalparikh vatsalparikh deleted the remove-am-mock-changeset branch June 26, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants