Skip to content

docs(ado-script): add missing ado-proxy and azure-wif-refresh bundles to inventory - #2207

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
docs/ado-script-bundle-inventory-fix-f809a1abd1ec7a1c
Draft

github-actions[bot] wants to merge 1 commit into
mainfrom
docs/ado-script-bundle-inventory-fix-f809a1abd1ec7a1c

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Agent Documentation Update

Findings

Area Issue File(s)
accuracy The "Workspace layout" tree and the release-workflow bundle-glob enumeration in docs/ado-script.md omitted the ado-proxy/ and azure-wif-refresh/ bundle directories (and the prepare-pr-base.js output was missing from the release-glob sentence), even though both bundles have their own dedicated documented sections earlier in the same file (ado-proxy.js at "ado-proxy: the same contract, applied to policy"; azure-wif-refresh.js in the bullet list of shipped bundles) and are treated as real shipped bundles by scripts/ado-script/src/__tests__/bundle-coverage.test.ts's NON_BUNDLE_DIRS guard and by .github/workflows/release.yml's generic ado-script/*.js glob. docs/ado-script.md

Applied Fixes

  • Added ado-proxy/ and azure-wif-refresh/ entries to the "Workspace layout" source tree, with brief descriptions consistent with the file's existing prose sections for these bundles.
  • Added ado-proxy.js and azure-wif-refresh.js to the ncc bundle output (gitignored) listing.
  • Added prepare-pr-base.js, ado-proxy.js, and azure-wif-refresh.js to the release-workflow bundle enumeration sentence so it lists every bundle the ado-script/*.js glob actually captures.

Notes

Verified against:

  • scripts/ado-script/src/ directory listing (confirms ado-proxy/ and azure-wif-refresh/ exist as real bundle source directories, each with their own index.ts).
  • scripts/ado-script/src/__tests__/bundle-coverage.test.ts's NON_BUNDLE_DIRS set (confirms these two directories are not excluded, i.e. they are treated as shipped bundles by the coverage guard).
  • .github/workflows/release.yml (confirms the release packaging step globs ado-script/*.js generically, so it already captures both .js outputs — only the documentation enumeration was stale).
  • The rest of docs/ado-script.md already has dedicated sections describing ado-proxy.js and azure-wif-refresh.js in detail, so this fix brings the workspace-layout summary and release-packaging description into sync with content that was already correct elsewhere in the same page.

No other documentation drift met the bar for inclusion in this change set; other candidate gaps found during the audit (e.g. container_runtime.rs missing from AGENTS.md's architecture tree, copilot-vendor ecosystem identifier missing from docs/network.md, mcp-author tool surface, reserved dependsOn/condition parameter names for target: job/target: stage) are already addressed by other open pull requests (#2198/#2203, #2202, #2201, #2190 respectively), so no changes were made for those to avoid duplication.


Created by the agent-documentation maintainer workflow.

Generated by Agent Documentation Maintainer · auto · 246.2 AIC · ⌖ 5.94 AIC · ⊞ 11.9K ·

… to inventory

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

This branch has not been deployed

No deployments
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.

0 participants