-
Notifications
You must be signed in to change notification settings - Fork 46
Add terser second pass for JavaScript formatting #6815
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...] |
|
🤖 SYSTEM_INIT: Smoke Copilot No Firewall ACTIVATED. PROCESSING pull request. ALL SUBSYSTEMS ONLINE. |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request... |
|
⚓ Avast! Smoke Copilot Safe Inputs be settin' sail on this pull request! 🏴☠️ |
|
Smoke Test Results Last 2 Merged PRs:
Test Results:
Available Tools: bash, grep, glob, view, create, edit, report_intent, github-* (list_pull_requests, get_file_contents, list_issues, etc.), safeoutputs-* (add_comment, add_labels, create_issue) Overall Status: PASS PR Author:
|
Smoke Test Results: Copilot Engine (No Firewall)Last 2 Merged PRs:
Test Results:
Status: PASS - All tests completed successfully.
|
Smoke Test Results (Run 20334667350)Last 2 Merged PRs:
Test Results:
Overall Status: PASS
|
Copilot Engine Smoke Test ResultsLast 2 Merged PRs:
Test Results:
Overall Status: PASS (4/5 tests passed - Serena result expected for Go project)
|
|
Smoke test results:
|
Adds terser minification after prettier to optimize .cjs files while preserving readability and TypeScript compatibility.
Changes
format:terserscript topkg/workflow/js/package.jsonformat:cjspipeline: prettier → terser → prettier-c dead_code=true,unused=true,collapse_vars=true --ecma 2022 --comments allExclusions
Files skipped from terser processing to preserve functionality:
// @ts-check,/// <reference)awaitor dynamicawait importImpact
Example
Before:
After terser + prettier:
Original prompt
💡 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.
Changeset
.cjsfiles afterprettier; addsterserdependency and updatesformat:cjspipeline. Files with TypeScript checks or top-level/dynamicawaitare excluded from terser processing to preserve behavior.