Skip to content

fix(ai-cu): surface actionable analysis failure diagnostics - #10857

Draft
Chien Yuan Chang (chienyuanchang) wants to merge 1 commit into
Azure:ai-cu/terminal-markdown-renderingfrom
chienyuanchang:fix/ai-cu-analysis-nested-diagnostics
Draft

Chien Yuan Chang (chienyuanchang) wants to merge 1 commit into
Azure:ai-cu/terminal-markdown-renderingfrom
chienyuanchang:fix/ai-cu-analysis-nested-diagnostics

Conversation

@chienyuanchang

Copy link
Copy Markdown
Member

Summary

  • Preserve sanitized service messages and nested error codes in terminal output and per-input reports for HTTP, polling, and inline failures.
  • Explain ModelNotSupported with the reported model name and commands to inspect analyzer support and resource model mappings; warn that defaults changes are resource-wide.
  • Show accurate success/failure/skip counts and identify failed input ordinals. Mention retained files only when successful results were actually saved.
  • Share bounded diagnostic redaction, including reflected credentials, and explicitly mark missing or truncated details. Keep cu-cli/analyze-report/v1 unchanged.

Stacking

Stacked on #10784; the base is ai-cu/terminal-markdown-rendering so this PR contains only the diagnostic fix. Retarget to main after the parent merges.

Validation

  • Full extension and nested REST-client go test -race ./... passed.
  • Builds, vet, modernization checks, strict lint, scoped spelling, and whitespace checks passed.
  • Regression coverage includes nested and unknown errors, missing details, all analysis output modes and reports, mixed batches, long outer diagnostics, and reflected API-key/Bearer credentials.
  • Offline fixtures/fake transports only; no live Azure analysis or resource mapping changes.

Analyzer-specific doctor readiness checks remain a separate follow-up.

Preserve sanitized service messages and nested causes in terminal and per-input report errors. Add model mapping remediation, accurate batch summaries, credential redaction and bounded diagnostics without changing the report schema.

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

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

@chienyuanchang
Chien Yuan Chang (chienyuanchang) marked this pull request as draft November 5, 2026 19:53
@github-actions github-actions Bot added the ext-content-understanding azure.ai.content-understanding extension label Nov 5, 2026

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

ext-content-understanding azure.ai.content-understanding extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant