Skip to content

strait test + strait events add - #8

Merged
bazingga08 merged 2 commits into
mainfrom
feat/cli-test-events-1105
Nov 5, 2026
Merged

bazingga08 merged 2 commits into
mainfrom
feat/cli-test-events-1105

Conversation

@bazingga08

Copy link
Copy Markdown
Contributor

Item cli-test-events-add. ADR: docs/adr/0001-strait-test-and-events-add.md.

  • strait test: one pass/fail list: engine answers (GET /healthz), the secret key is valid and which workspace, test or live (GET /v1/workspace), the workspace link domain serves AASA + assetlinks.json (existing runDoctor). Exit 1 on any failure, 2 for a missing/wrong-kind key, --json for CI. Key never printed; server text stripped of terminal control characters.
  • strait events add <name>: prints ready-to-paste event code per platform (react-native/expo, flutter, ios, android, web), detected from project files or --platform. Uses each SDK's real trackEvent; web uses reportWebOpen + POST /v1/event (as the web SDK's own example; it has no event helper).

Proof

  • npm test: 85 pass, 0 fail (2 existing skips).
  • Live: strait test against https://strait.link with the strait-dev key: 5/5 pass, exit 0; a fake key: Key fail + skips, exit 1.
  • Every snippet compiled against its SDK's origin/main: RN + web tsc --strict (typo control fails), Swift swift build, Flutter dart analyze (no issues; adding eventId: errors, so the Flutter snippet correctly omits it), Kotlin gradlew compileKotlin.

Not published: CHANGELOG "Unreleased"; ships with the next CLI release.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JfkGWSV4ar1AEGgGnCbGzd

bazingga08 and others added 2 commits November 5, 2026 16:24
strait test: one pass/fail list (engine, secret key with workspace and test/live,
link domain, AASA and assetlinks.json); exit 1 on failure, --json for CI.
strait events add <name>: ready-to-paste trackEvent code for React Native,
Flutter, iOS, Android and web, platform detected from the project files.
ADR docs/adr/0001. No publish: CHANGELOG Unreleased.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JfkGWSV4ar1AEGgGnCbGzd
…use extra arguments; README/help fixes from review

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JfkGWSV4ar1AEGgGnCbGzd
@bazingga08
bazingga08 merged commit 9102b7a into main Nov 5, 2026
4 checks 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.

1 participant