improve(javascript-pro): modernize runtime, tooling, and framework coverage#560
Merged
Conversation
…verage - Update persona from Node.js 20+ to Node.js 22 LTS / Node.js 24 LTS - Expand ES mastery from ES2023 to ES2025 (Object.groupBy, Promise.withResolvers, Set methods, Iterator helpers, RegExp.escape, explicit resource management) - Add Runtime ecosystem section covering Node.js 22/24, Bun, and Deno 2.x - Replace Jest-first testing with Vitest as default; add Playwright for E2E and node:test for library authors - Modernize Build and tooling: Vite 6/7/8 (Rolldown-powered) as default, Turbopack, Rolldown, esbuild; demote Webpack to legacy - Add Modern framework patterns section: React 19, Next.js 15, Vue 3.5/3.6, Svelte 5, SolidJS - Add Package management section: npm workspaces, pnpm, Bun, exports field, provenance - Expand Security practices with Socket.dev/Snyk, ignore-scripts, lockfile CI verification, typosquatting awareness - Update third description example to reference Vitest instead of Jest Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
Bot
added
the
review-pending
Contributor
👋 Thanks for contributing, @davila7!This PR touches What happens next
While you wait
This is an automated message. No action is required from you right now — a maintainer will review soon. |
Contributor
|
| Metric | Count |
|---|---|
| Total Components | 763 |
| ✅ Passed | 360 |
| ❌ Failed | 403 |
| 1006 |
❌ Failed Components (Top 5)
| Component | Errors | Warnings | Score |
|---|---|---|---|
vercel-edge-function |
3 | 4 | 81/100 |
prompt-engineer |
2 | 0 | 90/100 |
neon-expert |
2 | 2 | 88/100 |
agent-overview |
2 | 1 | 89/100 |
unused-code-cleaner |
2 | 1 | 89/100 |
...and 398 more failed component(s)
📊 View Full Report for detailed error messages and all components
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
2 issues found across 1 file
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="cli-tool/components/agents/programming-languages/javascript-pro.md">
<violation number="1" location="cli-tool/components/agents/programming-languages/javascript-pro.md:44">
P2: This line overstates Node 24 TypeScript type-stripping as stable; Node docs mark it as release-candidate in Node 24 and only mark it stable in Node 25.2+.</violation>
<violation number="2" location="cli-tool/components/agents/programming-languages/javascript-pro.md:132">
P2: `Vite 6/7/8 (Rolldown-powered)` is inaccurate: for Vite 6/7, Rolldown is an experimental opt-in package, while Vite 8 is the version with integrated Rolldown by default.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
|
|
||
| Runtime ecosystem: | ||
| - Node.js 22 LTS: stable Web Streams, built-in test runner, native fetch, improved ESM/CJS interop | ||
| - Node.js 24 LTS: native TypeScript type-stripping stable, updated V8 engine, performance improvements |
Contributor
There was a problem hiding this comment.
P2: This line overstates Node 24 TypeScript type-stripping as stable; Node docs mark it as release-candidate in Node 24 and only mark it stable in Node 25.2+.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At cli-tool/components/agents/programming-languages/javascript-pro.md, line 44:
<comment>This line overstates Node 24 TypeScript type-stripping as stable; Node docs mark it as release-candidate in Node 24 and only mark it stable in Node 25.2+.</comment>
<file context>
@@ -24,14 +24,27 @@ JavaScript development checklist:
+
+Runtime ecosystem:
+- Node.js 22 LTS: stable Web Streams, built-in test runner, native fetch, improved ESM/CJS interop
+- Node.js 24 LTS: native TypeScript type-stripping stable, updated V8 engine, performance improvements
+- Bun: 2-4x faster installs and execution, native TypeScript support, built-in bundler and test runner
+- Deno 2.x: npm-compatible, security-by-default with explicit permissions, excellent TypeScript support
</file context>
Suggested change
| - Node.js 24 LTS: native TypeScript type-stripping stable, updated V8 engine, performance improvements | |
| - Node.js 24 LTS: native TypeScript type-stripping enabled by default (release-candidate in v24), updated V8 engine, performance improvements |
| - Webpack optimization | ||
| - Rollup for libraries | ||
| - ESBuild integration | ||
| - Vite 6/7/8 (Rolldown-powered): default choice for new projects, 7x faster cold starts |
Contributor
There was a problem hiding this comment.
P2: Vite 6/7/8 (Rolldown-powered) is inaccurate: for Vite 6/7, Rolldown is an experimental opt-in package, while Vite 8 is the version with integrated Rolldown by default.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At cli-tool/components/agents/programming-languages/javascript-pro.md, line 132:
<comment>`Vite 6/7/8 (Rolldown-powered)` is inaccurate: for Vite 6/7, Rolldown is an experimental opt-in package, while Vite 8 is the version with integrated Rolldown by default.</comment>
<file context>
@@ -93,26 +108,46 @@ Browser API mastery:
-- Webpack optimization
-- Rollup for libraries
-- ESBuild integration
+- Vite 6/7/8 (Rolldown-powered): default choice for new projects, 7x faster cold starts
+- Turbopack: Next.js 15 default bundler, stable for production
+- Rolldown: standalone Rust-based bundler compatible with Rollup plugin ecosystem
</file context>
Suggested change
| - Vite 6/7/8 (Rolldown-powered): default choice for new projects, 7x faster cold starts | |
| - Vite 8 (Rolldown-powered): default choice for new projects; for Vite 6/7, Rolldown is available via the experimental rolldown-vite package |
davila7
added a commit
that referenced
this pull request
May 2, 2026
Reflects merged improvements to cli-tool/components/agents/programming-languages/javascript-pro.md. Automated by pr-verification cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated Component Improvement
Component:
cli-tool/components/agents/programming-languages/javascript-pro.mdDownloads: 3,837 total | 299/week (2nd most downloaded agent)
Changes
Runtime version baseline updated — Persona upgraded from
Node.js 20+toNode.js 22 LTS / Node.js 24 LTS. ES mastery updated from ES2023 to ES2025 with new features:Object.groupBy(),Promise.withResolvers(), Set methods, Iterator helpers,RegExp.escape(), explicit resource management (using/await using).New Runtime ecosystem section — Added coverage of Node.js 22/24 key additions (native fetch, built-in test runner, native TypeScript type-stripping in Node 24), Bun (2-4x performance, native TS), and Deno 2.x (npm-compatible, security-by-default) with guidance on when to choose each.
Vitest promoted as default test runner — Restructured Testing methodology: Vitest leads for new projects (native ESM/TS, 10-20x faster), Jest retained for legacy/React Native, Playwright added for E2E,
node:testfor library authors.Build tooling modernized — Vite 6/7/8 (Rolldown-powered) becomes the default recommendation. Added Turbopack (Next.js 15 stable), Rolldown (standalone), esbuild. Webpack demoted to legacy/maintenance-mode.
Modern framework patterns section added — React 19 (Compiler, Server Components, Actions API,
use()hook), Next.js 15 (Turbopack, partial prerendering), Vue 3.5/3.6 (Vapor Mode), Svelte 5 (Runes), SolidJS.New Package management section — npm workspaces +
--provenance, pnpm strict isolation, Bun fastest installs,package.jsonexports field for ESM/CJS dual publishing,ignore-scripts=true, lockfile CI verification.Security section expanded for supply-chain threats — Added Socket.dev/Snyk behavioral analysis,
ignore-scripts=truein.npmrc, lockfile verification in CI, typosquatting awareness (per CISA Sep 2025 alert).Description example updated — Third example now references Vitest instead of Jest for consistency with the improved system prompt.
Research Summary
The agent was directing users toward approaching end-of-life runtimes (Node.js 20 EOL April 2026 is close), outdated tooling (Webpack is maintenance-mode, Vite now 17M weekly downloads), and superseded test runner defaults (Vitest exceeds Jest in 2025 State of JS satisfaction). With 3,837 downloads and ~299/week, stale guidance here had significant impact.
Validation
name,description,tools)javascript-pro.mdprogramming-languagesAutomated review cycle by Component Improvement Loop
Summary by cubic
Modernizes the javascript-pro component to Node.js 22/24 and ES2025, and updates guidance across runtimes, testing, bundling, and frameworks. Keeps users on current, faster, and more secure defaults.
cli-tool/components/); updatedagents/programming-languages/javascript-pro.md. No new components; catalog (docs/components.json) unchanged; no new environment variables or secrets.Bun, andDeno 2.x.Vitest;Jestfor legacy/React Native;Playwrightfor E2E;node:testfor library authors.Vite6/7/8 (Rolldown) as default;Turbopackfor Next.js 15;Rolldown;esbuild;Webpackmarked legacy.npm/pnpm/Bunguidance (--provenance,exports,ignore-scripts=true, lockfile verification); supply-chain checks with Socket.dev/Snyk.Written for commit 7488375. Summary will update on new commits.