Apply bundler to noop.cjs to resolve load_agent_output dependency

[Copilot]

The noop.cjs script requires load_agent_output.cjs but was embedded directly without bundling, causing unresolved dependencies at runtime.

Changes

• scripts.go: Added bundled getNoOpScript() following pattern used by notify_comment_error and log parsers

  • Embedded source as noopScriptSource
  • Added noopScriptBundledOnce sync.Once for lazy bundling
  • Inlines load_agent_output.cjs via BundleJavaScriptFromSources()

• noop.go: Removed unbundled getNoOpScript(), now calls bundled version from scripts.go

• js.go: Removed unused noopScript variable

• noop_bundling_test.go: Added test verifying require statements removed and loadAgentOutput inlined

Pattern

Follows existing bundler pattern:

func getNoOpScript() string {
    noopScriptBundledOnce.Do(func() {
        sources := GetJavaScriptSources()
        bundled, err := BundleJavaScriptFromSources(noopScriptSource, sources, "")
        // ...
    })
    return noopScriptBundled
}

Compiled workflows now contain inlined loadAgentOutput function with no require statements.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• https://api.github.com/user
  • Triggering command: gh api user --jq .login (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

Make sure the bundler is applied to the no_op JavaScript

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[pelikhan]

@copilot lint and fix tests

[Copilot]

Pull Request Overview

This PR fixes a runtime dependency issue where noop.cjs required load_agent_output.cjs but was embedded without bundling. The fix applies the existing JavaScript bundler pattern to inline the dependency.

Key changes:

• Applied bundler to noop.cjs to inline load_agent_output.cjs dependency
• Added test to verify bundling removes require statements
• Cleaned up duplicate/unused code references

Reviewed Changes

Copilot reviewed 74 out of 74 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
pkg/workflow/scripts.go	Added bundled getNoOpScript() with lazy bundling via sync.Once
pkg/workflow/noop_bundling_test.go	Added test verifying require removal and function inlining
pkg/workflow/noop.go	Removed unbundled getNoOpScript() (now in scripts.go)
pkg/workflow/js.go	Removed unused noopScript variable
.github/workflows/*.lock.yml (64 files)	Compiled workflows now contain inlined loadAgentOutput function

---

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