[WIP] Fix failing GitHub Actions job JS Tests (shard 4/4)

[Copilot]

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Fix the failing GitHub Actions job "JS Tests (shard 4/4)"
Analyze the Actions logs, identify the root cause of the failure, and implement a fix.
Check run ID: 76410111696
Job URL: https://github.com/github/gh-aw/actions/runs/25995861865/job/76410111696

---

pr-sous-chef: requested branch update (run: https://github.com/github/gh-aw/actions/runs/25996524238)

Generated by 👨‍🍳 PR Sous Chef · ◷

[Copilot]

Pull request overview

This PR refactors safe-output large-content handling and applies it to create_issue so oversized issue fields are written to files instead of directly appended.

Changes:

• Extracted large string detection/offloading into maybeHandleLargeContent.
• Reused the helper in the default safe-output handler.
• Added large-content handling to createIssueHandler.

Show a summary per file

File	Description
actions/setup/js/safe_outputs_handlers.cjs	Refactors large-content handling and applies it to create issue safe outputs.

Copilot's findings

Tip

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

Comments suppressed due to low confidence (1)

actions/setup/js/safe_outputs_handlers.cjs:52

• The helper stops after offloading the first oversized string field, so any additional >16k-token string fields remain in the NDJSON entry that gets appended. For tools like create_issue with both title and body (or dynamic tools with multiple large string inputs), this can still write an oversized safe-output record and reproduce the failure this handler is meant to avoid. The scan should continue and replace every oversized string field before appending.

    for (const [key, value] of Object.entries(entry)) {
      if (typeof value === "string") {
        const tokens = estimateTokens(value);
        if (tokens > TOKEN_THRESHOLD) {
          largeContent = value;
          largeFieldName = key;
          server.debug(`Field '${key}' has ${tokens} tokens (exceeds ${TOKEN_THRESHOLD})`);
          break;

• Files reviewed: 1/1 changed files
• Comments generated: 1

[pelikhan]

@copilot review all comments

[Copilot]

@copilot review all comments

Addressed. I reviewed the thread and applied the requested follow-up so duplicate create_issue entries also offload large content before append, with focused test coverage for that path. Latest commit: 9e558f9.

[github-actions]

WIP: tests failing on JS shard 4/4 — please review.
``
@copilot review all comments

Generated by 👨‍🍳 PR Sous Chef · ◷