[PR #1083] [MERGED] fix(artifacts): keep integrity policy and extraction claims aligned #2631

New issue

Closed

opened 2026-10-17 21:08:26 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-17 21:08:26 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/1083
Author: @morluto
Created: 10/8/2026
Status: ✅ Merged
Merged: 10/8/2026
Merged by: @morluto

Base: feat/artifact-integrity-policy ← Head: cursor/integrity-policy-cleanup-d298

📝 Commits (2)

aee942a fix(artifacts): keep integrity policy and extraction claims aligned
dd1fe03 test(artifacts): type the nested extraction fixture as a directory path

📊 Changes

3 files changed (+76 additions, -4 deletions)

View changed files

📝 src/application/ArtifactExtraction.ts (+17 -3)
📝 src/cli/coreBinaryCommands.ts (+13 -1)
➕ tests/boundary/filesystem/artifactExtractionNestedIntegrity.test.ts (+46 -0)

📄 Description

Summary

Follow-up to #902.

rea analyze routed a JavaScript application without the integrity policy, so the remediation flag the error recommends was rejected on that command. The routed command now accepts --integrity-policy and forwards it.
Extraction copied every inventory contradiction into the result and said those files were written. Only materialized occurrences stay in that list. Nested contradictions are named as unwritten.

The skill-version comment is already stale: this branch and its merge base are both version 27.

Verification

npm run test:focused -- tests/boundary/filesystem/artifactExtractionNestedIntegrity.test.ts tests/boundary/mcp/artifactMcp.test.ts
Pre-push npm run check:fast

Real Hopper/Ghidra verification was not applicable.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/1083 **Author:** [@morluto](https://github.com/morluto) **Created:** 10/8/2026 **Status:** ✅ Merged **Merged:** 10/8/2026 **Merged by:** [@morluto](https://github.com/morluto) **Base:** `feat/artifact-integrity-policy` ← **Head:** `cursor/integrity-policy-cleanup-d298` --- ### 📝 Commits (2) - [`aee942a`](https://github.com/morluto/rea/commit/aee942a41dc378aa602acd15d70e1e33b32afd0a) fix(artifacts): keep integrity policy and extraction claims aligned - [`dd1fe03`](https://github.com/morluto/rea/commit/dd1fe03c68b622a883e5508f1b2d108ecd17e400) test(artifacts): type the nested extraction fixture as a directory path ### 📊 Changes **3 files changed** (+76 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/application/ArtifactExtraction.ts` (+17 -3) 📝 `src/cli/coreBinaryCommands.ts` (+13 -1) ➕ `tests/boundary/filesystem/artifactExtractionNestedIntegrity.test.ts` (+46 -0) </details> ### 📄 Description  ## Summary Follow-up to #902. - `rea analyze` routed a JavaScript application without the integrity policy, so the remediation flag the error recommends was rejected on that command. The routed command now accepts `--integrity-policy` and forwards it. - Extraction copied every inventory contradiction into the result and said those files were written. Only materialized occurrences stay in that list. Nested contradictions are named as unwritten. The skill-version comment is already stale: this branch and its merge base are both version 27. ## Verification - `npm run test:focused -- tests/boundary/filesystem/artifactExtractionNestedIntegrity.test.ts tests/boundary/mcp/artifactMcp.test.ts` - Pre-push `npm run check:fast` Real Hopper/Ghidra verification was not applicable.  <div><a href="https://cursor.com/agents/bc-b50166c8-2750-4725-adf5-16ed0e97d298?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a href="https://cursor.com/background-agent?bcId=bc-b50166c8-2750-4725-adf5-16ed0e97d298&cursor_ref=pr_footer&cursor_cta=open_in_cursor"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a> </div> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-17 21:08:26 +00:00