Skip to content

Harden validation and add cloud CI#1

Draft
Eldorado-ling wants to merge 2 commits into
mainfrom
agent/cloud-validate-v1-1
Draft

Harden validation and add cloud CI#1
Eldorado-ling wants to merge 2 commits into
mainfrom
agent/cloud-validate-v1-1

Conversation

@Eldorado-ling

Copy link
Copy Markdown
Owner

What changed

  • keep OpenCode skill metadata within the discovery limit
  • accept current and legacy OpenCode share URL forms
  • fail closed on malformed inbox paths, trust settings, repo mismatches, and Git command failures
  • add command timeouts and correct threat-model wording
  • add standard-library unit and real local Git inbox integration tests
  • add Linux/Windows GitHub Actions coverage for Python 3.8 and 3.13

Why

The original v1.0.1 had no automated tests or CI, its skill description exceeded OpenCode's documented limit, and several malformed-input and failure cases were not rejected deterministically.

Validation

  • python -m unittest discover -s tests -v (17 tests)
  • python -m py_compile verify_inbox.py tests/test_verify_inbox.py
  • skill quick validation
  • OpenCode 1.17.13 discovery check

This is a draft PR created to validate the candidate in GitHub Actions before any merge or release.

@Eldorado-ling

Copy link
Copy Markdown
Owner Author

Cloud validation passed for commit 2226e3d:

  • Ubuntu: Python 3.8 ✅, Python 3.13 ✅
  • Windows: Python 3.8 ✅, Python 3.13 ✅
  • 17 tests per job, including a real temporary Git inbox pipeline
  • py_compile passed in every job
  • live GitHub commit API check passed: Tier 4 author/committer attribution ✅ and Tier 5 signature verification ✅

Run: https://github.com/Eldorado-ling/opencode-handoff-p2p/actions/runs/29149781331

Not covered by this automated run: fetching a real private OpenCode share transcript and a two-account collaborator handoff.

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.

1 participant