Skip to content

Fix self-test output on macOS and Linux#87

Merged
corv89 merged 2 commits intomainfrom
fix-selftest-output
Jan 7, 2026
Merged

Fix self-test output on macOS and Linux#87
corv89 merged 2 commits intomainfrom
fix-selftest-output

Conversation

@corv89
Copy link
Owner

@corv89 corv89 commented Jan 7, 2026

Summary

  • Change test script from platform.node() to sys.version_info (avoids triggering uname -p subprocess on macOS)
  • Fix output parsing to get first script line, not last summary line
  • Only show output on failure, not success (cleaner status display)

Problem

Running shannot status showed jarring output:

  • macOS: Created pending session for uname -p, showed "Run 'shannot approve'..."
  • Linux: Showed "*** No commands or writes were queued. ***"

After

Runtime:
  ✓ Stdlib: ~/.local/share/shannot/runtime
  ✓ Sandbox binary: ~/.local/share/shannot/runtime/pypy3-c
  ✓ Self-test: passed (78ms)

- Change test script from platform.node() to sys.version_info
  (avoids triggering uname -p subprocess on macOS)
- Fix output parsing to get first script line, not last summary line
- Only show output on failure, not success (cleaner status display)
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

CHANGELOG.md is out of date

Please update the changelog by running:

make changelog

Or manually:

git-cliff --config cliff.toml -o CHANGELOG.md

Then commit the updated CHANGELOG.md file.

@corv89 corv89 self-assigned this Jan 7, 2026
@corv89 corv89 added bug Something isn't working enhancement New feature or request labels Jan 7, 2026
@corv89 corv89 merged commit 84f20d2 into main Jan 7, 2026
14 checks passed
@corv89 corv89 deleted the fix-selftest-output branch January 7, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant