Skip to content

Fix debug message typo, replace process.exit with throws, fix test fixture naming#31

Draft
Copilot wants to merge 3 commits into
feat/support-windowsfrom
copilot/fix-test-issues-and-bugs
Draft

Fix debug message typo, replace process.exit with throws, fix test fixture naming#31
Copilot wants to merge 3 commits into
feat/support-windowsfrom
copilot/fix-test-issues-and-bugs

Conversation

Copilot AI commented Feb 27, 2026

Copy link
Copy Markdown

Code review fixes across source and tests.

  • src/zinc.ts — Remove extra )} in error debug template string producing garbled output
  • src/utils.ts — Replace process.exit(-1) with proper error propagation so callers can handle failures; include supported content types in error message
  • tests/utils/fixtures.ts, tests/engine.test.ts — Fix typo engineMartixengineMatrix

Before (utils.ts):

} else {
  debug(`Unsupported content type: ${contentType}`);
  process.exit(-1);
}
} catch (err) {
  debug(`error when downloading and extracting the binary file: ${err}`);
  process.exit(-1);
}

After:

} else {
  throw new Error(`Unsupported content type: ${contentType}. Supported types: application/gzip, application/x-gzip, application/octet-stream, application/zip`);
}
} catch (err) {
  debug(`error when downloading and extracting the binary file: ${err}`);
  throw err;
}

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 27, 2026 15:53
… engineMartix→engineMatrix

Co-authored-by: Blankll <28639911+Blankll@users.noreply.github.com>
Co-authored-by: Blankll <28639911+Blankll@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix test issues and address detected bugs Fix debug message typo, replace process.exit with throws, fix test fixture naming Feb 27, 2026
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.

2 participants