Skip to content

feat(ci): add StackBlitz templates for e2e apps#540

Open
ryanbas21 wants to merge 1 commit into
mainfrom
feat/stackblitz-templates
Open

feat(ci): add StackBlitz templates for e2e apps#540
ryanbas21 wants to merge 1 commit into
mainfrom
feat/stackblitz-templates

Conversation

@ryanbas21

@ryanbas21 ryanbas21 commented Feb 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds --template './e2e/*-app' to pkg-pr-new so PR comments include interactive StackBlitz links for all e2e apps (davinci, journey, oidc, device-client, protect)
  • Switches to --pnpm flag for correct pnpm pack behavior in the monorepo
  • Keeps --packageManager=pnpm so PR comments show pnpm add install commands

Test plan

  • Verify the PR CI comment includes StackBlitz "Open in StackBlitz" links for each e2e app
  • Confirm the StackBlitz instances resolve snapshot package versions correctly
  • Verify pnpm pack is used (check CI logs for pack output)

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow publishing configuration to refine package selection and publishing process parameters.

@changeset-bot

changeset-bot Bot commented Feb 26, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e2cb9cc

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 no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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 Feb 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@ryanbas21, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 57 minutes. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: cb4d44de-55a6-4e11-8e94-444484b03067

📥 Commits

Reviewing files that changed from the base of the PR and between 5615d50 and e2cb9cc.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml
📝 Walkthrough

Walkthrough

The CI workflow's publish step was updated to include two additional arguments: --template './e2e/*-app' and --pnpm when invoking pnpm pkg-pr-new publish. This refines package selection and explicitly enables pnpm in the publishing process.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/ci.yml
Updated the publish step to pass --template './e2e/*-app' and --pnpm arguments to pnpm pkg-pr-new, specifying template patterns and explicitly enabling pnpm for package publishing.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • cerebrl
  • ancheetah

Poem

🐰 A hop and a skip, the workflow now glides,
With templates and flags to guide the release tides,
pnpm enchanted, the publish runs fast,
Our e2e packages blast forward at last! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding StackBlitz templates for e2e apps to the CI workflow.
Description check ✅ Passed The description comprehensively covers the changes, includes a clear summary of modifications, and provides a concrete test plan for verification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/stackblitz-templates

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 and usage tips.

@nx-cloud

nx-cloud Bot commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit e2cb9cc

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

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


☁️ Nx Cloud last updated this comment at 2026-06-10 17:11:47 UTC

@pkg-pr-new

pkg-pr-new Bot commented Feb 26, 2026

Copy link
Copy Markdown
More templates

@forgerock/davinci-client

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

@forgerock/device-client

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

@forgerock/journey-client

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

@forgerock/oidc-client

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

@forgerock/protect

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

@forgerock/sdk-types

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

@forgerock/sdk-utilities

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

@forgerock/iframe-manager

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

@forgerock/sdk-logger

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

@forgerock/sdk-oidc

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

@forgerock/sdk-request-middleware

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

@forgerock/storage

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

commit: e2cb9cc

@github-actions

github-actions Bot commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Deployed cec06d9 to https://ForgeRock.github.io/ping-javascript-sdk/pr-540/cec06d93ce561efe90b5a278471a6b82f76a4c20 branch gh-pages in ForgeRock/ping-javascript-sdk

@github-actions

github-actions Bot commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

📦 Bundle Size Analysis

📦 Bundle Size Analysis

🆕 New Packages

🆕 @forgerock/protect - 144.6 KB (new)
🆕 @forgerock/sdk-types - 7.9 KB (new)
🆕 @forgerock/device-client - 10.0 KB (new)
🆕 @forgerock/device-client - 0.0 KB (new)
🆕 @forgerock/iframe-manager - 2.4 KB (new)
🆕 @forgerock/storage - 1.5 KB (new)
🆕 @forgerock/sdk-request-middleware - 4.6 KB (new)
🆕 @forgerock/sdk-logger - 1.6 KB (new)
🆕 @forgerock/sdk-oidc - 5.7 KB (new)
🆕 @forgerock/journey-client - 93.0 KB (new)
🆕 @forgerock/journey-client - 0.0 KB (new)
🆕 @forgerock/sdk-utilities - 11.2 KB (new)
🆕 @forgerock/davinci-client - 53.8 KB (new)
🆕 @forgerock/oidc-client - 30.5 KB (new)


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

@codecov-commenter

codecov-commenter commented Feb 27, 2026

Copy link
Copy Markdown

Codecov Report

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

❌ Your project status has failed because the head coverage (21.20%) 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     #540      +/-   ##
==========================================
+ Coverage   18.07%   21.20%   +3.13%     
==========================================
  Files         155      154       -1     
  Lines       24398    24838     +440     
  Branches     1203     1401     +198     
==========================================
+ Hits         4410     5268     +858     
+ Misses      19988    19570     -418     

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.

@ancheetah

Copy link
Copy Markdown
Collaborator

How are the templates useful? Is the idea so that we can copy paste the package.json into our e2e locally so we can run the test suites? Also, the templates generated by Stackblitz don't seem to boot up the application in the browser. There might be some error I'm not seeing.

@ryanbas21 ryanbas21 force-pushed the feat/stackblitz-templates branch from 5615d50 to f528f61 Compare May 12, 2026 20:01

@nx-cloud nx-cloud Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.

Nx Cloud has identified a possible root cause for your failed CI:

We determined these e2e failures are unrelated to this PR's changes, which only affect CI publishing configuration. The same failures are reproducible in branch 576, confirming a pre-existing issue with the external PingAM/ForgeRock OAuth2 service returning invalid_request errors for all authentication flows. The affected projects (@forgerock/protect-suites, @forgerock/oidc-suites) were not touched by this PR.

No code changes were suggested for this issue.

Trigger a rerun:

Rerun CI

Nx Cloud View detailed reasoning on Nx Cloud ↗


🎓 Learn more about Self-Healing CI on nx.dev

@ryanbas21 ryanbas21 force-pushed the feat/stackblitz-templates branch from f528f61 to 33d5a85 Compare May 19, 2026 21:12
Adds --template flag to pkg-pr-new so PR comments include
interactive StackBlitz links for all e2e apps. Also switches
to --pnpm for correct packing in the monorepo.
@ryanbas21 ryanbas21 force-pushed the feat/stackblitz-templates branch from 33d5a85 to e2cb9cc Compare June 10, 2026 17:02
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