[GH-ISSUE #3459] inspect_artifact inventory evidence reads only the first substep #1296

Closed
opened 2026-10-16 21:03:52 +00:00 by bsakaguchi · 0 comments
Owner

Originally created by @sx4im on GitHub (Oct 14, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/3459

parseArtifactInventoryEvidence in src/domain/artifactInventoryEvidence.ts (~line 64) resolves the inspect_artifact branch with .substeps[0]?.evidence. When an inspection carries multiple retained substeps, everything after the first is silently dropped before artifact comparison.

This feeds compareArtifacts and the android/apple application parsers, so unlike the completion-only drop fixed in #3292, this one changes results: identifiers found in later substeps never reach the comparison.

Spotted during review of #3292 (thanks rahul05ranjan for catching it). Filing so it doesn't get lost.

Originally created by @sx4im on GitHub (Oct 14, 2026). Original GitHub issue: https://github.com/morluto/rea/issues/3459 `parseArtifactInventoryEvidence` in `src/domain/artifactInventoryEvidence.ts` (~line 64) resolves the `inspect_artifact` branch with `.substeps[0]?.evidence`. When an inspection carries multiple retained substeps, everything after the first is silently dropped before artifact comparison. This feeds `compareArtifacts` and the android/apple application parsers, so unlike the completion-only drop fixed in #3292, this one changes results: identifiers found in later substeps never reach the comparison. Spotted during review of #3292 (thanks rahul05ranjan for catching it). Filing so it doesn't get lost.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
morluto/rea#1296
No description provided.