Skip to content

fix(ado-script): flatten bundled artifacts to top-level gate.js/import.js#671

Merged
jamesadevine merged 1 commit into
mainfrom
copilot/flatten-ado-script-directory
May 20, 2026
Merged

fix(ado-script): flatten bundled artifacts to top-level gate.js/import.js#671
jamesadevine merged 1 commit into
mainfrom
copilot/flatten-ado-script-directory

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 20, 2026

Summary

This change removes the scripts/ado-script/dist/*/index.js runtime contract and switches the system to flat artifact names (gate.js, import.js) inside ado-script.zip. Compiler-emitted pipeline steps now execute those files directly by name.

  • Build/output layout

    • Updated scripts/ado-script/package.json build scripts to emit flat gate.js / import.js artifacts (no dist/ output contract).
    • Updated ignore rules to match the new artifact/temp layout.
  • Runtime wiring

    • Updated AdoScriptExtension runtime paths to:
      • /tmp/ado-aw-scripts/ado-script/gate.js
      • /tmp/ado-aw-scripts/ado-script/import.js
    • Updated release packaging to include flat files directly in ado-script.zip.
  • Verification surface updates

    • Updated Rust tests, TS smoke tests, and compiler assertions that previously referenced dist/.../index.js.
    • Updated docs (repo docs + site docs) to reflect the flat artifact contract.
const GATE_EVAL_PATH: &str = "/tmp/ado-aw-scripts/ado-script/gate.js";
pub(crate) const IMPORT_EVAL_PATH: &str = "/tmp/ado-aw-scripts/ado-script/import.js";

Test plan

  • cargo test
  • cargo clippy --all-targets --all-features
  • cd scripts/ado-script && npm run build && npm run typecheck && npm test && npm run test:smoke

Agent-Logs-Url: https://github.com/githubnext/ado-aw/sessions/1c1a805e-f12b-48fa-ad8b-3ddcbfef73f6

Co-authored-by: jamesadevine <4742697+jamesadevine@users.noreply.github.com>
@jamesadevine jamesadevine marked this pull request as ready for review May 20, 2026 14:03
@jamesadevine jamesadevine merged commit 2b02d0d into main May 20, 2026
20 checks passed
@jamesadevine jamesadevine deleted the copilot/flatten-ado-script-directory branch May 20, 2026 23:06
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