Skip to content

feat(shadictl): add git snapshot artifacts#10

Open
muscariello wants to merge 8 commits intomainfrom
7-git-backed-file-change-tracking-for-sandboxed-runs
Open

feat(shadictl): add git snapshot artifacts#10
muscariello wants to merge 8 commits intomainfrom
7-git-backed-file-change-tracking-for-sandboxed-runs

Conversation

@muscariello
Copy link
Member

@muscariello muscariello commented Mar 11, 2026

Summary

  • add opt-in git snapshot capture around sandboxed runs
  • persist snapshots in a stable runs/<artifact_id>/snapshot.json layout
  • include Git state hashes and before/after comparison fields for downstream tooling

Testing

  • cargo test -p shadictl

Stacks on #9.

@codecov-commenter
Copy link

codecov-commenter commented Mar 11, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 90.46392% with 74 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
crates/shadictl/src/main.rs 90.46% 74 Missing ⚠️

📢 Thoughts on this report? Let us know!

@muscariello muscariello requested review from a team and msardara March 11, 2026 08:10
@muscariello
feat(shadictl): add git snapshot artifacts
43e9333 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello
docs: add git snapshot documentation
4d2a947 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello
test(shadictl): make snapshot coverage portable
2597a36 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello
test(shadictl): stabilize windows snapshot coverage
1f342c3 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello
test(shadictl): relax windows snapshot assertions
2733a5d 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello
feat(shadictl): track nested git repos in snapshots
457c45c 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello muscariello force-pushed the 7-git-backed-file-change-tracking-for-sandboxed-runs branch from b88c3fd to 457c45c Compare March 11, 2026 14:26
@muscariello muscariello changed the base branch from 6-rework-the-docs-landing-page-and-navigation to main March 11, 2026 14:27
@muscariello muscariello linked an issue Mar 11, 2026 that may be closed by this pull request
Git-backed file change tracking for sandboxed runs #7
Open
@muscariello
fix(shadictl): close nested snapshot test handle on windows
a98a758 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello
chore: fix code coverage workflow
b93df60 
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
msardara
msardara approved these changes Mar 11, 2026
View reviewed changes
Copy link
Member

@msardara msardara left a comment

Choose a reason for hiding this comment

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

LGTM ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@msardara msardara msardara approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Git-backed file change tracking for sandboxed runs

3 participants

@muscariello @codecov-commenter @msardara