Skip to content

docs(network): document missing copilot-vendor ecosystem identifier - #2145

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
docs/network-copilot-vendor-ecosystem-512fffbc094a8525
Draft

github-actions[bot] wants to merge 1 commit into
mainfrom
docs/network-copilot-vendor-ecosystem-512fffbc094a8525

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Agent Documentation Update

Findings

Area Issue File(s)
accuracy copilot-vendor is a real, usable network.allowed ecosystem identifier defined in src/data/ecosystem_domains.json (resolved the same way as every other identifier via get_ecosystem_domains), but it was entirely missing from both the primary ecosystem-identifier table and the additional-ecosystems list in docs/network.md. Agents authoring network: blocks had no way to discover this identifier from the docs. docs/network.md

Applied Fixes

  • Added copilot-vendor to the ecosystem-identifier table in docs/network.md, describing its four domains (api.business.githubcopilot.com, api.enterprise.githubcopilot.com, api.individual.githubcopilot.com, telemetry.enterprise.githubcopilot.com)

Notes

Verified by diffing every key in src/data/ecosystem_domains.json against every identifier documented in docs/network.md's two ecosystem tables (main + additional). copilot-vendor was the only key present in the JSON but absent from the docs; every other documented identifier matched the code exactly (including the single compound identifier, default-safe-outputs). Also spot-checked docs/schedule-syntax.md against src/fuzzy_schedule.rs (hour intervals, weekday aliases) and docs/cli.md against src/main.rs's Commands enum, both of which were already accurate. No other files were changed.


Created by the agent-documentation maintainer workflow.

Generated by Agent Documentation Maintainer · auto · 108.7 AIC · ⌖ 19.2 AIC · ⊞ 12.4K ·

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.

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