Add Disassembler Availability report to diff output - #44
Merged
Conversation
At startup, probe all candidate IL disassemblers (dotnet-ildasm, ilspycmd) for availability and version, then display the results as a table in the Markdown, HTML, and audit log report headers. This makes it explicit which tools were available/unavailable during a run, improving the explainability of IL comparison confidence levels. - Add DisassemblerProbeResult record model - Add DisassemblerHelper.ProbeAllCandidates() probing logic - Add FileDiffResultLists.DisassemblerAvailability property - Add availability table to Markdown report header (HeaderSectionWriter) - Add availability table to HTML report header (AppendHeaderSection) - Add disassemblerAvailability array to audit log JSON - Add AuditLogDisassemblerAvailability model - Add 9 new tests across 4 test classes - Update doc/samples (diff_report.md, diff_report.html, audit_log.json) - Update DEVELOPER_GUIDE.md, TESTING_GUIDE.md (EN/JP bilingual) - Update CHANGELOG.md (EN/JP), README.md (EN/JP) https://claude.ai/code/session_01RN1uYPHDtxJL13wpqCeVwt
- CS1739: Remove named parameter syntax from WaitForExit (positional only) - CA1031: Replace bare catch with specific InvalidOperationException catch https://claude.ai/code/session_01RN1uYPHDtxJL13wpqCeVwt
Apply TH_BG_DEFAULT (#fafafa) to <th> elements in the Disassembler Availability table, matching the Ignored Files table header style. https://claude.ai/code/session_01RN1uYPHDtxJL13wpqCeVwt
Apply background:#fafafa to <th> elements in the sample report to match the code change. https://claude.ai/code/session_01RN1uYPHDtxJL13wpqCeVwt
…y table Add text-align:center to the Available <td> cells in the HTML report and update the sample HTML accordingly. The MD report already uses :---------: for center alignment. https://claude.ai/code/session_01RN1uYPHDtxJL13wpqCeVwt
Widthdom
changed the title
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
2 participants
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
Changes
DisassemblerProbeResultmodel,DisassemblerHelper.ProbeDisassemblers()ReportGenerateService,HtmlReportGenerateService,AuditLogGenerateServicediff_report.md,diff_report.html,audit_log.json)