Skip to content

Fix reporting of failed analysis operations - #27

Open
Changjian Wang (changjian-wang) wants to merge 1 commit into
mainfrom
fix/39540887-report-failed-analysis
Open

Changjian Wang (changjian-wang) wants to merge 1 commit into
mainfrom
fix/39540887-report-failed-analysis

Conversation

@changjian-wang

Copy link
Copy Markdown
Member

Summary

  • detect raw analysis envelopes whose final service status is Failed
  • convert nested service errors into the existing structured ServiceError model
  • cover LRO/inline and binary/URL analysis paths
  • verify CLI failures return exit code 1, write a failed report entry, and do not write a result file

This fixes Azure DevOps Bug 39540887 without adding an extension allowlist. The service remains the source of truth for supported content formats.

Validation

  • core copyright header check, Ruff, mypy, and all unit tests pass
  • standalone Ruff, mypy, core unit tests, remaining unit tests, and offline integration tests pass
  • tests/unit/test_public_docs.py::test_pypi_readme_links_to_product_readme cannot pass in this Windows checkout because packages/standalone/README.md was checked out as a regular file with core.symlinks=false; this is unrelated to the change
  • the repository ci.sh has CRLF line endings in this checkout, so equivalent CI commands were run directly with .venv\Scripts\python.exe

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