Skip to content

fix(security): harden artifact source identity - #167

Merged
Eclipseic1848 merged 1 commit into
mainfrom
codex/issue-160-artifact-path-security
Sep 3, 2026
Merged

fix(security): harden artifact source identity#167
Eclipseic1848 merged 1 commit into
mainfrom
codex/issue-160-artifact-path-security

Conversation

@Eclipseic1848

Copy link
Copy Markdown
Owner

Summary

  • reject leaf symlinks and linked/nested temporary output paths before Artifact import
  • pin reads to an opened FileHandle and verify file identity before reading/deleting
  • preserve valid output capture, redaction, hashing, and source cleanup
  • synchronize the bundled Python Worker

Verification

  • red test reproduced the junction escape on Windows
  • npm workspace Artifact tests: 9 passed, 1 platform skip
  • CoreMind Tools project: 71 passed, 3 skipped
  • Python Node parity: 6 passed
  • all workspace typechecks passed
  • Biome lint passed

Closes #160

@Eclipseic1848
Eclipseic1848 merged commit c940bbc into main Sep 3, 2026
2 checks passed
@Eclipseic1848
Eclipseic1848 deleted the codex/issue-160-artifact-path-security branch September 3, 2026 14:24
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.

security: prevent symlink escape when importing tool artifacts

1 participant