Skip to content

Use Linux sandbox annotations - #30

Merged
foxfirecodes merged 1 commit into
mainfrom
lew/linux-sandbox-annotations-tmp
Aug 27, 2026
Merged

foxfirecodes merged 1 commit into
mainfrom
lew/linux-sandbox-annotations-tmp

Conversation

@foxfirecodes

@foxfirecodes foxfirecodes commented Jun 19, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • integrate the published Linux sandbox annotation support from http://github.com/foxfirecodes/sandbox-runtime
  • store the exact ASRT command used for annotation lookup, including mixed review-only segments
  • add Linux annotation noise filtering, filesystem retry-reason extraction, and a unique no-op marker to avoid ASRT command-key collisions

Tests

  • npm run typecheck
  • npm test
  • PI_AUTO_SANDBOX_E2E=1 npx vitest run tests/sandbox-e2e.test.ts (run outside pi-auto sandbox; passed per local validation)

Stacked on #27.

@yonilerner yonilerner left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure if this needs to be stacked on your other experimental one but im fine merging both tbh, seems like they work

@foxfirecodes

Copy link
Copy Markdown
Collaborator Author

not sure if this needs to be stacked on your other experimental one but im fine merging both tbh, seems like they work

@yonilerner mainly just stacked cuz this PR supports mixed review only commands. both need more testing before merge but the mixed command sequencing has worked flawlessly for me so far so I think it'll be fine to ship soon

@foxfirecodes
foxfirecodes force-pushed the lew/linux-sandbox-annotations-tmp branch from 9914f49 to b937ee9 Compare June 21, 2026 17:10
@foxfirecodes
foxfirecodes force-pushed the lew/linux-sandbox-annotations-tmp branch from b937ee9 to 26fd42e Compare August 27, 2026 04:30
Base automatically changed from review-only-mixed-routing-2 to main August 27, 2026 04:31
@foxfirecodes
foxfirecodes force-pushed the lew/linux-sandbox-annotations-tmp branch from 26fd42e to 64916cf Compare August 27, 2026 04:32
@foxfirecodes
foxfirecodes merged commit 5bd419c into main Aug 27, 2026
1 check passed
@foxfirecodes
foxfirecodes deleted the lew/linux-sandbox-annotations-tmp branch August 27, 2026 04:32
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.

2 participants