docs: post-#252 reconciliation for azure_ai/agents/<id> (init prompt + targets docs) [blocked on #253] - #256
Closed
tangym wants to merge 2 commits into
Closed
docs: post-#252 reconciliation for azure_ai/agents/<id> (init prompt + targets docs) [blocked on #253]#256tangym wants to merge 2 commits into
tangym wants to merge 2 commits into
Conversation
PR #252 added the azure_ai/agents/<AGENT_ID> target route for hosted Foundry agents and rejects target.tools / target.system_prompt for it at parse time. The auth surfaces (docs/getting-started.md, docs/config/schema.md) were updated, but the assert-ai init system prompt did not learn about the new route, so init currently proposes configs that fail at parse time when a user describes a Foundry agent. - Split the 'model' decision-tree branch into 2a (hosted chat model) and 2b (Foundry hosted agent). - Forbid target.system_prompt and target.tools in the 2b sub-case to match the TargetConfig parser rejection. - Tighten the Target-Type propose-readiness check. - Note the azure_ai/agents/* sub-case in the inference YAML skeleton and in the customization hints. Parse-error prevention; no v1/v2 functional dependency.
Add the azure_ai/agents/<AGENT_ID> route to the user-facing target docs: a row in the decision tree and 'paths at a glance' tables in docs/targets/README.md, plus a new 'Foundry hosted agent target' subsection in docs/targets/model-and-tools.md with a runnable YAML snippet and auth requirements. This commit is intentionally HELD until #253 ships full Foundry v2 agent support (UUID IDs). PR #252 only covers v1 agents, and the v2 path still requires a workaround, so promoting this route in public target docs now would oversell coverage. When #253 lands, rebase this commit on top, drop or amend as needed (e.g. tweak wording to cover both v1 and v2), and flip the PR to ready for review.
2 tasks
github-actions
Bot
requested review from
AaronAspinwall123,
changliu2,
jakepresent and
minthigpen
June 27, 2026 07:11
Collaborator
|
Looks stale, I will close for hygiene, please reopen if needed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reconciles the
assert-ai initsystem prompt and the user-facing target docs with theazure_ai/agents/<AGENT_ID>Foundry hosted-agent route that #252 added.Splits cleanly into two commits so the safe part can ship independently if needed:
docs(prompts): teach assert-ai init about azure_ai/agents/<id>— Parse-error prevention. TheTargetConfigparser rejectstarget.toolsandtarget.system_promptwhen the model name starts withazure_ai/agents/, but the init system prompt did not learn this rule, soassert-ai initcurrently proposes configs that fail at parse time when a user describes a Foundry agent. v1/v2-agnostic — would be correct to ship today.docs(targets): document Foundry hosted-agent target [BLOCKED on #253]— Promotes the Foundry route indocs/targets/README.md(decision-tree + "paths at a glance" tables) and adds a "Foundry hosted agent target" subsection todocs/targets/model-and-tools.md. Held until feat(targets): add native target for Azure AI Foundry v2 Agents (UUID IDs) #253 ships full Foundry v2 agent support. PR feat(azure-auth): native AAD support for azure_ai/* targets (Foundry hosted agents) #252 only covers v1, and the v2 path still needs a workaround, so promoting the route in public target docs now would oversell coverage.Why draft?
Bidirectional pickup for #253:
When #253 lands, the v2 implementer should:
If we want commit 1 sooner, it can be cherry-picked into a tiny standalone ship-now PR without losing the draft.
Verification
src/my_agent/bot.py" → routes tocallable. ✓asst_abc123" → routes tomodelsub-case 2b; does not ask forsystem_prompt/tools. ✓python3 -m compileall -q assert_ai(no code changed; sanity check).Files
assert_ai/internal_pipeline_prompts/init_system.mddocs/targets/README.mddocs/targets/model-and-tools.mdRelated: #252, #253