feat(read-snapshot): supercharge — _short_id, duration_ms, exit_code, 35 new tests
- Add _short_id(): sha256:<12-hex> (19 chars), prefix always preserved - Fix text format: snapshot_id[:12] → _short_id() — was only 5 hex after prefix - Add duration_ms (float, 3dp) and exit_code (0) to all JSON output paths: full manifest, --no-manifest, --path-prefix, combined --no-manifest + --path-prefix - Update _SnapshotOutput TypedDict: add duration_ms, exit_code fields with docstring documenting file_count semantics under --path-prefix filtering - Expand run() docstring: document --no-manifest + --path-prefix combination, file_count reflects filtered count, updated text format spec - Add tests/test_read_snapshot_supercharge.py: 35 tests covering _short_id unit, duration/exit_code on all output paths, text format short-ID, --no-manifest + --path-prefix flag interaction, data integrity (sha256: prefix, unicode paths, timezone-aware created_at), path prefix edge cases (empty prefix, no trailing slash, exact filename, traversal attempt), performance thresholds - Fix test_cmd_read_snapshot.py: bare-hex missing-snapshot test now uses sha256: prefix so it reaches the store lookup instead of validation rejection
0 comments
muse hub commit comment sha256:46466579ba1357bcf85f888fe7d09a099e9bf7a134b7f3cc64599fe399877d86 --body "your comment"
No comments yet. Be the first to start the discussion.