Skip to content

feat: scaffold read-only sourceosctl CLI#2

Merged
mdheller merged 4 commits intomainfrom
copilot/add-sourceosctl-cli-scaffold
Apr 30, 2026
Merged

feat: scaffold read-only sourceosctl CLI#2
mdheller merged 4 commits intomainfrom
copilot/add-sourceosctl-cli-scaffold

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 30, 2026

  • Create sourceosctl/ Python package with CLI entry point (cli.py)
  • Implement commands: doctor, profiles list, nlboot evidence inspect <path>, release inspect <path>, fingerprint collect --dry-run, ai labs list, agents sandbox plan --dry-run
  • All commands are read-only or dry-run; mutating paths rejected with exit code 1
  • Add bin/sourceosctl entry-point script (no install required)
  • Add fixtures/sample_nlboot_evidence.json and fixtures/sample_release.json
  • Add tests/test_cli.py — 21 unit tests covering all commands and error paths
  • Add scripts/validate_scaffold.py — fixes GNU Make 4.3 heredoc incompatibility
  • Update Makefilevalidate now depends on test; test target added
  • Update README.md — CLI usage table and running instructions added
  • Add/extend .gitignore — excludes __pycache__, build artifacts, .DS_Store, *.tmp
  • make validate passes locally (21/21 tests OK, scaffold validation OK)
  • Code review: no comments; CodeQL: 0 alerts
  • No-op commit pushed to trigger fresh CI check run after workflow approval

Copilot AI linked an issue Apr 30, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add read-only sourceosctl CLI scaffold feat: scaffold read-only sourceosctl CLI Apr 30, 2026
Copilot AI requested a review from mdheller April 30, 2026 13:25
Copy link
Copy Markdown
Contributor

@copilot Scope looks correct for issue #1, but this PR is still draft and the validation workflow is stuck at action_required. Please mark the PR ready for review and refresh/rebase or push a no-op commit if needed so checks rerun under current repo settings. Keep scope unchanged and include/retain validation evidence.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 30, 2026

@copilot Scope looks correct for issue #1, but this PR is still draft and the validation workflow is stuck at action_required. Please mark the PR ready for review and refresh/rebase or push a no-op ...

Pushed commit e6824de (extends .gitignore) to trigger a fresh CI run. Scope is unchanged — make validate passes locally (21/21 tests, scaffold check OK).

Marking the PR ready for review requires a maintainer to click "Ready for review" in the GitHub UI (or approve the workflow run if the action_required status is a first-run protection gate). The commit push will kick off new check runs once the workflow is approved.

@mdheller mdheller marked this pull request as ready for review April 30, 2026 13:40
Copy link
Copy Markdown
Contributor

@mdheller mdheller left a comment

Choose a reason for hiding this comment

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

Approved. Scope matches issue #1: read-only/dry-run sourceosctl scaffold, fixtures, tests, README updates, and validation wiring. CI is green and no mutating behavior is introduced.

@mdheller mdheller merged commit 7161d5b into main Apr 30, 2026
1 check passed
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.

Agent task: scaffold read-only sourceosctl CLI

2 participants