Skip to content

fix: ADO rendering fixes, dismissal status, and docs overhaul - #24

Merged
dsiddharth2 merged 2 commits into
mainfrom
fix/dismissal-reply-status
May 26, 2026
Merged

fix: ADO rendering fixes, dismissal status, and docs overhaul#24
dsiddharth2 merged 2 commits into
mainfrom
fix/dismissal-reply-status

Conversation

@dsiddharth2

Copy link
Copy Markdown
Owner

Summary

  • Dismissal thread status: PostFixReplyActivity now honors the status parameter — dismissed findings resolve as WONT_FIX instead of incorrectly using FIXED
  • ADO work item auto-linking: All PR ID references (#6718) wrapped in backticks to prevent ADO from rendering them as linked PBIs/Bugs
  • Code suggestion rendering: Added newline before code fences in suggestions so ADO renders proper code blocks instead of inline backtick text
  • README slimmed: 472 → 131 lines — moved detailed content to docs/, kept essentials
  • Architecture docs rewritten: Reflects actual current codebase (batch orchestration, two-pass review, risk classifier, dismissals, smart diff, sliding window)
  • Feature docs updated: Dismissal handling in fix-verification.md, ADO rendering notes in post-findings.md

Test plan

  • All 68 unit tests pass
  • Verify ADO PR summary no longer auto-links PR IDs to work items
  • Verify code suggestion blocks render as formatted code in ADO comments
  • Verify dismissed findings resolve threads with WONT_FIX status

@
fix(reply): honor status parameter for dismissal thread resolution

The caller passes WONT_FIX for dismissed findings but execute()
was hardcoded to FIXED — dismissed threads were incorrectly
marked as fixed instead of wont-fix.
- Wrap PR ID references in backticks to prevent ADO work item auto-linking
- Add newline before code fences in suggestions for proper ADO rendering
- Slim README from 472 to 131 lines, move details to docs/
- Rewrite architecture.md to reflect current codebase (batch orchestration,
  two-pass review, risk classifier, dismissals, smart diff, sliding window)
- Document dismissal handling and ADO rendering notes in feature docs
@dsiddharth2
dsiddharth2 merged commit 69f4a96 into main May 26, 2026
3 checks passed
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