Skip to content

chore(sdk-python): Wave 1 hygiene — arch doc sync - #5

Merged
Raphjacksun7 merged 3 commits into
mainfrom
chore/wave1-hygiene
Jun 10, 2026
Merged

Raphjacksun7 merged 3 commits into
mainfrom
chore/wave1-hygiene

Conversation

@Raphjacksun7

Copy link
Copy Markdown
Contributor

Summary

  • Synced docs/architecture/PocketStation-v2.3.md:
    • Renamed `ps` command`pks` command throughout
    • Updated DHAT allocation claim (counting allocator gate; full DHAT deferred)
    • Added protocol note in §14.6 Phase 2 entry

Note: no FAKE_SCAFFOLD_INVENTORY.md root duplicate or PHASE*.md files at root; only arch doc sync applies.

STAFF_REVIEW

  1. Purpose: Housekeeping — fix stale arch claims.
  2. Files changed: docs/architecture/PocketStation-v2.3.md (3 edits).
  3. Architecture boundary affected: None.
  4. Behavior preserved: No code changed.
  5. Tests: N/A.
  6. Mocked/stubbed: Unchanged.
  7. Public API change: None.
  8. Claims: Corrected DHAT and pspks.
  9. Scaffold inventory: No change.
  10. Final decision: PASS

Raphael Avocegamou added 3 commits June 8, 2026 17:30
…tighten gitignore

station.py:
- Extract URL/duration magic values into named module constants
- Add PocketStation.create() classmethod for one-call room+connect setup
- Replace bare r.raise_for_status() with typed PocketStationError raises
  (network_error, http_error, parse_error codes) for structured handling
- Rename stop() to close() — Pythonic async resource convention
- Remove misleading "Falls back gracefully" docstring from listen()

__init__.py:
- Reorder imports: PocketStation (primary class) before PocketStationClient

gitignore: add Python artifact patterns (__pycache__, .egg-info, .pytest_cache, venv)

All 15 tests pass.
@Raphjacksun7
Raphjacksun7 merged commit dab72d1 into main Jun 10, 2026
6 checks passed
@Raphjacksun7
Raphjacksun7 deleted the chore/wave1-hygiene branch June 10, 2026 00:11
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