Fix CI failures across deep-forest, paypal-components, mint-components, and program-boilerplate by Copilot · Pull Request #487 · saasquatch/program-tools

Copilot · 2026-03-23T21:50:27Z

Downgrade puppeteer from ^22.10.0 to ~20.9.0 (last major with executionContext() API that Stencil 2.4 needs)
Remove testing-related portions of @stencil+core+2.4.0.patch (keep only the compiler URL constructor fix)
Run npm install to regenerate package-lock.json
Run tests — all 96 pass (5 suites)
Code review — no issues with our changes (false positives from base branch diffs)
Security check — no vulnerabilities in puppeteer@20.9.0

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…nges, update test expectations Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com> Agent-Logs-Url: https://github.com/saasquatch/program-tools/sessions/1809d8b3-6ed1-48db-afa7-bbc3069d1d70

…-components tests Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com> Agent-Logs-Url: https://github.com/saasquatch/program-tools/sessions/1809d8b3-6ed1-48db-afa7-bbc3069d1d70

… big-stat test expectations Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com> Agent-Logs-Url: https://github.com/saasquatch/program-tools/sessions/0521da34-fc0c-4d21-8bd9-9571715572a3

Copilot

Pull request overview

This PR aims to restore green CI across multiple packages by updating test infrastructure/tooling to be compatible with newer Node/Puppeteer/Jest behavior and aligning test expectations with current component/runtime output.

Changes:

Added/updated Jest typing and updated several program-boilerplate tests to match current error messages/throw matchers.
Updated mint-components Stencil e2e setup (Chromium args), adjusted sqm-big-stat e2e expectations, and introduced a patch-package patch for @stencil/core@2.4.0 to remove Puppeteer executionContext() usage.
Updated the deep-forest workflow to use newer GitHub Actions and Node 20; refreshed paypal-components lockfile to newer dependency resolutions.

Reviewed changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
packages/program-boilerplate/package.json	Adds `@types/jest` to align TS/Jest typing with Jest v30.
packages/program-boilerplate/package-lock.json	Locks `@types/jest@30` and associated dependency metadata changes.
packages/program-boilerplate/tests/webtask.test.ts	Updates expected error payload to include trigger type in the error message.
packages/program-boilerplate/tests/utils.test.ts	Casts test bodies to `any` to satisfy stricter typings when calling `getTriggerSchema`.
packages/program-boilerplate/tests/trigger.test.ts	Updates expected error messages and adds multiple `as any` casts for test fixtures.
packages/program-boilerplate/tests/transaction.test.ts	Updates test fixtures/types and expected query variables to match current transaction behavior.
packages/program-boilerplate/tests/jsonata.test.ts	Replaces deprecated `toThrowError`/`not.toThrowError` with `toThrow`/`not.toThrow`.
packages/paypal-components/package-lock.json	Updates lockfile contents (package version + dependency resolution changes).
packages/mint-components/stencil.config.ts	Adds Puppeteer/Chromium args for Stencil testing runs.
packages/mint-components/src/components/sqm-big-stat/sqm-big-stat.e2e.ts	Updates invalid-stat expectations from `!!!` to `-`.
packages/mint-components/patches/StencilBot (@Stencil)+core+2.4.0.patch	Patch-package patch to replace `executionContext()` calls with ElementHandle `evaluate()`/`evaluateHandle()`.
.github/workflows/deep-forest.yml	Bumps checkout/setup-node actions to v4 and Node to 20.x.

Files not reviewed (2)

packages/paypal-components/package-lock.json: Language not supported
packages/program-boilerplate/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-23T23:00:28Z

packages/program-boilerplate/__tests__/trigger.test.ts

        PROGRAM_INTROSPECTION: spy,
      };
-      const result = triggerProgram(testBody, spyingProgram);
+      const result = triggerProgram(testBody as any, spyingProgram);


Using as any for the triggerProgram request body in these tests sidesteps the public ProgramTriggerBody / ProgramIntrospectionBody contracts and can let breaking type changes slip through. Prefer constructing typed fixtures (or a factory helper) so the tests continue to validate the expected request shape as types evolve.

Copilot · 2026-03-23T23:00:28Z

packages/program-boilerplate/__tests__/trigger.test.ts

      messageType: "PROGRAM_TRIGGER" as "PROGRAM_TRIGGER",
      ids: ["123", "456"],
-      activeTrigger: { type: "AFTER_USER_EVENT_PROCESSED" },
+      activeTrigger: { type: "AFTER_USER_EVENT_PROCESSED" as const, time: 0, user: { id: "test", accountId: "test", programGoals: [] } },


This inline activeTrigger literal is very long and hard to read/maintain. Consider formatting it across multiple lines and/or extracting a shared typed fixture builder for ProgramTriggerBody.activeTrigger so future schema changes are localized to one place.

Copilot · 2026-03-23T23:00:29Z

packages/program-boilerplate/__tests__/transaction.test.ts

+  const testUser: any = {
    id: "reffererID",
    accountId: "reffererACCOUNTID",
+    programGoals: [] as any[],
    referredByReferral: {
      id: "referralID",
    },
  };
-  let transaction = new Transaction(testContext);
+  let transaction = new Transaction(testContext as any);


Introducing any for testUser (and testContext as any below) reduces the usefulness of these tests as a guardrail for the User/ProgramTriggerBody shapes. Prefer creating testUser/testContext objects that satisfy the real types (or use a typed factory) so compile-time checks catch schema drift.

Copilot · 2026-03-23T23:00:29Z

packages/mint-components/stencil.config.ts

 export const config: Config = {
  namespace: "mint-components",
+  testing: {
+    browserArgs: ["--no-sandbox", "--disable-setuid-sandbox"],


testing.browserArgs disables Chromium’s sandbox for all test runs. To avoid weakening local test security unnecessarily, consider only adding these flags when running in CI (e.g., conditional on process.env.CI) or making them configurable via an env var with a secure default.

Suggested change

export const config: Config = {

namespace: "mint-components",

testing: {

browserArgs: ["--no-sandbox", "--disable-setuid-sandbox"],

const browserArgs =

process.env.CI ? ["--no-sandbox", "--disable-setuid-sandbox"] : undefined;

export const config: Config = {

namespace: "mint-components",

testing: {

...(browserArgs ? { browserArgs } : {}),

Copilot · 2026-03-23T23:00:29Z

packages/program-boilerplate/__tests__/utils.test.ts

    ];

-    expect(getTriggerSchema(programTriggerBody)).toStrictEqual(expectedOutput);
+    expect(getTriggerSchema(programTriggerBody as any)).toStrictEqual(expectedOutput);


Casting the trigger body to any here (and in the other getTriggerSchema tests) bypasses type safety and can hide contract drift as ProgramTriggerBody evolves. Prefer building a minimal valid ProgramTriggerBody fixture (e.g., provide program: {id, rules, templateId} and a user with required fields) or use a typed helper to construct test bodies.

Downgrade puppeteer from ^22.10.0 to ~20.9.0 since @stencil/core@2.4.0 uses executionContext() which was removed in puppeteer v21.1.1. This eliminates the need to patch @stencil/core's minified testing code. The compiler URL constructor fix (the original patch) is retained. Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com> Agent-Logs-Url: https://github.com/saasquatch/program-tools/sessions/c456bd4e-c3ad-4779-9879-2778f8f89627

Initial plan

ae315e9

Copilot AI assigned Copilot and Coleton (Locrian24) Mar 23, 2026

Copilot started work on behalf of Coleton (Locrian24) March 23, 2026 21:50 View session

Copilot AI and others added 2 commits March 23, 2026 22:07

fix: program-boilerplate tests - add @types/jest, fix Jest 30 API cha…

acdcaca

…nges, update test expectations Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com> Agent-Logs-Url: https://github.com/saasquatch/program-tools/sessions/1809d8b3-6ed1-48db-afa7-bbc3069d1d70

fix: update paypal-components lock file and add --no-sandbox for mint…

a2540cf

…-components tests Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com> Agent-Logs-Url: https://github.com/saasquatch/program-tools/sessions/1809d8b3-6ed1-48db-afa7-bbc3069d1d70

Copilot AI changed the title ~~[WIP] Fix issues causing CI job failures for Deep Forest Sync~~ Fix CI failures across deep-forest, paypal-components, mint-components, and program-boilerplate Mar 23, 2026

Copilot AI requested a review from Coleton (Locrian24) March 23, 2026 22:14

Copilot finished work on behalf of Coleton (Locrian24) March 23, 2026 22:14

Copilot started work on behalf of Coleton (Locrian24) March 23, 2026 22:20 View session

fix: patch Stencil testing for Puppeteer v22 compatibility and update…

e689612

… big-stat test expectations Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com> Agent-Logs-Url: https://github.com/saasquatch/program-tools/sessions/0521da34-fc0c-4d21-8bd9-9571715572a3

Copilot finished work on behalf of Coleton (Locrian24) March 23, 2026 22:38

Coleton (Locrian24) marked this pull request as ready for review March 23, 2026 22:56

Copilot AI review requested due to automatic review settings March 23, 2026 22:56

Copilot started reviewing on behalf of Coleton (Locrian24) March 23, 2026 22:57 View session

Copilot AI reviewed Mar 23, 2026

View reviewed changes

Copilot started work on behalf of Coleton (Locrian24) March 24, 2026 20:31 View session

Copilot finished work on behalf of Coleton (Locrian24) March 24, 2026 20:34

Copilot started work on behalf of Coleton (Locrian24) March 24, 2026 20:35 View session

Copilot finished work on behalf of Coleton (Locrian24) March 24, 2026 20:54

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix CI failures across deep-forest, paypal-components, mint-components, and program-boilerplate#487

Fix CI failures across deep-forest, paypal-components, mint-components, and program-boilerplate#487
Copilot wants to merge 5 commits intomasterfrom
copilot/fix-ci-job-deep-forest-sync

Copilot AI commented Mar 23, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 23, 2026

Uh oh!

Copilot AI Mar 23, 2026

Uh oh!

Copilot AI Mar 23, 2026

Uh oh!

Copilot AI Mar 23, 2026

Uh oh!

Copilot AI Mar 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

3 participants

Conversation

Copilot AI commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Mar 23, 2026 •

edited

Loading