Skip to content

Execute project tests - #928

Open
TorinAsakura wants to merge 10 commits into
pre-v3/architecture-foundationfrom
feat/project-test-execution
Open

Execute project tests#928
TorinAsakura wants to merge 10 commits into
pre-v3/architecture-foundationfrom
feat/project-test-execution

Conversation

@TorinAsakura

@TorinAsakura TorinAsakura commented Aug 27, 2026

Copy link
Copy Markdown
Member

Task

How to verify

Main scenario

  1. Context: a project uses the checked Yarn runtime with Plug and Play and the ESM loader
    Action: run yarn test, yarn test unit, and yarn test integration
    Expected result: each route executes through the public Node.js node:test run() API, consumes the native TestsStream, selects the expected files, and completes with the native test result

Additional scenario

  1. Context: a project test fails or a watched run receives its AbortSignal
    Action: consume the project result through the test command or GitHub checks
    Expected result: failures preserve the native test and stderr events, watch abort completes the native stream, and no managed child, IPC protocol, forced termination policy, or project lifecycle status is involved

Proofs

  • Root runtime contract: Node.js >=24.15.0 <25; GitHub Actions reads it through node-version-file: package.json
  • Behavioral baseline on Node.js 24.19.0 before the structural-only follow-up: source-focused matrix 15 passed, checked-runtime matrix 15 passed, 0 failed
  • Structural follow-up only relocates existing discovery, runtime argv resolution, scenario policy, and command input types; execution logic, public commands, and plugin-checks source remain unchanged
  • Current-head commit hooks: formatting, scoped typecheck, and scoped lint passed
  • Checked runtime and dist runtime are byte-identical with SHA-256 a43146939b398835bccf6e683b2fc7708ca6410c2ab661d8007874b19c038613
  • Runtime inventory matches generated documentation: 38 commands and 44 plugins
  • Localization synchronization: passed
  • Normal help exposes exactly yarn test, yarn test unit, and yarn test integration; command definitions contain no private producer route
  • Behavioral baseline disposable packaged PnP/ESM consumer: unit 1 passed, integration 1 passed, general 2 passed, outer commands exited 0

@TorinAsakura TorinAsakura self-assigned this Aug 27, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-08-29T03:39:27.967175Z b79d9a5 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 33a15ec73f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread yarn/raijin/src/infrastructure/adapters/node/execution/executor.ts Outdated
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.

1 participant