fix: global install fails with ERR_MODULE_NOT_FOUND for workspace packages

[khaliqgant]

Summary

• Fix verify-publish workflow to build packages before npm pack on PRs
• Add workspace package symlink setup in postinstall.js for global installs

Problem

When globally installing agent-relay, the CLI fails with:

Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@agent-relay/daemon' imported from dist/src/cli/index.js

This happens because:

1. The verify-publish workflow was running npm ci --ignore-scripts then npm pack without building
2. Without building, the packages/*/dist/ folders don't exist
3. When globally installed, Node.js can't resolve the workspace package imports

Solution

1. Remove --ignore-scripts from npm ci in verify-publish workflow
2. Add explicit npm run build step before npm pack on PRs
3. Add setupWorkspacePackageLinks() in postinstall.js as a fallback to create symlinks from node_modules/@agent-relay/* to packages/* for bundled/global installs

Test plan

• PR workflow verify-publish passes
• Global install test: npm install -g agent-relay then agent-relay --version
• npx test: npx agent-relay --version

🤖 Generated with Claude Code

---

[devin-ai-integration]

Devin Review found 1 potential issue.

View issue and 5 additional flags in Devin Review.

[devin-ai-integration]

🔴 verify job doesn't use local tarball on PRs, testing npm package instead of PR changes

The verify job's Get package spec step (lines 61-73) doesn't check for pull_request event and always sets SPEC="agent-relay" (the npm package), ignoring the local tarball that was just built.

Click to expand

Problem

In the verify job, the workflow:

1. Builds and packs the local code with npm pack (line 54)
2. But then the Get package spec step only checks inputs.version and defaults to SPEC="agent-relay" from npm

Compare with the verify-macos job at .github/workflows/verify-publish.yml:346-361 which correctly handles PRs:

if [ "${{ github.event_name }}" = "pull_request" ]; then
  SPEC="$(pwd)/$(ls agent-relay-*.tgz | head -1)"
else
  ...
fi

Impact

On PRs, the verify job tests the published npm package instead of the PR's changes. This means:

• The PR's fix for global install (setupWorkspacePackageLinks) is never tested
• The workflow passes even if the PR introduces bugs
• False confidence that PR changes work correctly

Expected vs Actual

• Expected: On PRs, test the local tarball built from PR code
• Actual: On PRs, test the existing npm package agent-relay@latest

(Refers to lines 61-73)

Recommendation: Add a check for pull_request event similar to the verify-macos job:

if [ "${{ github.event_name }}" = "pull_request" ]; then
  SPEC="$(pwd)/$(ls agent-relay-*.tgz | head -1)"
elif [ -z "$VERSION_INPUT" ] || [ "$VERSION_INPUT" = "latest" ]; then
  ...

---

Was this helpful? React with 👍 or 👎 to provide feedback.