Skip to content

fix: bump js-yaml to 4.3.2 (supersedes #266) - #291

Merged
CaseyKelly merged 1 commit into
mainfrom
claude/fantasyphish-pr-266-dr1fyb
Sep 24, 2026
Merged

CaseyKelly merged 1 commit into
mainfrom
claude/fantasyphish-pr-266-dr1fyb

Conversation

@CaseyKelly

Copy link
Copy Markdown
Owner

Same one-commit lockfile change as Dependabot's #266 (js-yaml 4.3.1 → 4.3.2, a dev-only transitive dependency; the release backports merge-sequence limits for the CPU-usage issue in nodeca/js-yaml#797).

Why a separate PR

#266 can't go green as-is. GitHub gives Dependabot-triggered workflow runs no Actions secrets, so NEON_API_KEY / TEST_DATABASE_URL / TEST_DIRECT_URL are empty there. No Neon branch gets created, prisma migrate deploy falls back to localhost:5432 (P1001), and test, lint-typecheck-build and analyze all fail. Re-running doesn't help because re-runs keep Dependabot's permissions. This branch isn't pushed by Dependabot, so its runs get the normal secrets.

Once this merges, Dependabot should close #266 as superseded.

Verification

  • npm ci, npm run typecheck, npm run lint pass locally
  • js-yaml resolves to 4.3.2

🤖 Generated with Claude Code

https://claude.ai/code/session_01FYAwNpBcL88R2Ks8s5GR7X


Generated by Claude Code

Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.3.1 to 4.3.2.
- [Changelog](https://github.com/nodeca/js-yaml/blob/4.3.2/CHANGELOG.md)
- [Commits](nodeca/js-yaml@4.3.1...4.3.2)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 4.3.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@vercel

vercel Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
fantasyphish Ready Ready Preview Sep 24, 2026 1:13am UTC

Request Review

@CaseyKelly
CaseyKelly marked this pull request as ready for review September 24, 2026 01:13
Copilot AI lite review requested due to automatic review settings September 24, 2026 01:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

Copy link
Copy Markdown

📦 Next.js Bundle Analysis for fantasyphish

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@github-actions

Copy link
Copy Markdown

🎭 Playwright test results

✅ 38 passed, 0 failed, 4 flaky, 1 skipped (43 total)

Status Test Location
⚠️ Flaky should 404 for a logged-in non-admin user admin-shows.spec.ts:14
⚠️ Flaky should 404 for a logged-in non-admin user admin-usage.spec.ts:14
⚠️ Flaky should allow admin to delete submissions admin.spec.ts:6
⚠️ Flaky pick page should have no automatically detectable accessibility violations picker.spec.ts:7

Full HTML report with traces, screenshots, and console logs: download the playwright-results artifact from this run and open it with npx playwright show-report (or npm run test:trace locally).

@CaseyKelly
CaseyKelly merged commit 867d4ec into main Sep 24, 2026
8 checks passed
@CaseyKelly
CaseyKelly deleted the claude/fantasyphish-pr-266-dr1fyb branch September 24, 2026 01:18

This branch was successfully deployed

1 active deployment
Preview — b35faa1a Deployed Sep 24, 2026 by vercel[bot]
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.

2 participants