Skip to content

docs(site): document exec-context-pr.js bundle in ado-script reference#908

Merged
jamesadevine merged 1 commit into
mainfrom
docs/ado-script-exec-context-pr-55077ff9b49b7766
Jun 8, 2026
Merged

docs(site): document exec-context-pr.js bundle in ado-script reference#908
jamesadevine merged 1 commit into
mainfrom
docs/ado-script-exec-context-pr-55077ff9b49b7766

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Developers reading ado-script.mdx now see all three bundles (gate.js, import.js, exec-context-pr.js) instead of just two
  • The new section explains what exec-context-pr.js does, how merge-base resolution works, the trust boundary, and its full env-var contract
  • The workspace layout tree, release-workflow paragraph, and wiring table are updated to reflect the real workspace structure

Changes

  • site/src/content/docs/reference/ado-script.mdx
    • Updated page description and intro to list all three bundles
    • Added ## What exec-context-pr.js does section with merge-base strategy, trust-boundary notes, and env-var contract table
    • Updated workspace layout tree to include exec-context-pr/ source directory and exec-context-pr.js build output
    • Updated release-workflow paragraph to mention exec-context-pr.js
    • Renamed ### Agent job wiring subsection to reflect dual consumers (import.js and exec-context-pr.js)
    • Expanded ### What gets emitted, by case table with an on.pr w/ exec-context column (8-row matrix)
    • Added note pointing to exec-context-pr.js as a real working example in ### Add a new bundle
    • Added Execution Context to ## See also

Accuracy checks

  • Bundle existence and source layout confirmed against scripts/ado-script/src/exec-context-pr/
  • Env-var contract verified against src/compile/extensions/exec_context/pr.rs::prepare_step() and scripts/ado-script/src/exec-context-pr/validate.ts
  • Merge-base resolution paths confirmed against scripts/ado-script/src/exec-context-pr/merge-base.ts
  • Activation condition (on.pr + execution-context.pr.enabled != false) verified against src/compile/extensions/exec_context/pr.rs::should_activate()
  • Trust-boundary description verified against src/compile/extensions/exec_context/pr.rs module docstring and git.ts::bearerEnv
  • Wiring ownership confirmed against src/compile/extensions/ado_script.rs::prepare_steps() (installs bundle when exec_context_pr_active)

Validation

  • cd site && npm ci && npm run build — passed, all 34 internal links valid

Created by the docs-writer workflow.

Generated by Docs Writer · sonnet46 12.4M ·

Add the missing third ado-script bundle to the reference page.
The exec-context-pr.js bundle has been shipping in ado-script.zip
since it was introduced for execution-context PR support, but the
ado-script.mdx docs only mentioned gate.js and import.js.

Changes:
- Update intro paragraph to list all three bundles
- Add '## What exec-context-pr.js does' section with merge-base
  resolution strategy, trust-boundary notes, and env-var contract table
- Update workspace layout tree to include exec-context-pr/ directory
  and exec-context-pr.js build output
- Update release-workflow paragraph to list exec-context-pr.js
- Rename 'Agent job' wiring section to reflect two consumers
  (import.js and exec-context-pr.js)
- Expand 'What gets emitted' table with on.pr/exec-context column
- Add exec-context-pr.js as a real working example in 'Add a new bundle'
- Add Execution Context to See also

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine jamesadevine marked this pull request as ready for review June 8, 2026 19:02
@jamesadevine jamesadevine merged commit 245c2d8 into main Jun 8, 2026
@jamesadevine jamesadevine deleted the docs/ado-script-exec-context-pr-55077ff9b49b7766 branch June 8, 2026 19:02
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.

1 participant