Skip to content

Add GitHub action#68

Closed
alan-george-lk wants to merge 6 commits into
mainfrom
feat/github-action
Closed

Add GitHub action#68
alan-george-lk wants to merge 6 commits into
mainfrom
feat/github-action

Conversation

@alan-george-lk

@alan-george-lk alan-george-lk commented Jun 22, 2026

Copy link
Copy Markdown
  • Adds GitHub action for client SDKs to validate basic token source operations against
  • Adds optional env TOKEN_SERVER_PORT for deconflicting port numbers with existing processes if needed, otherwise falls back to existing 3000 default
  • Fixes build warnings
  • Update README with changes/GitHub action instructions

alan-george-lk and others added 5 commits June 22, 2026 13:21
TypeScript 6.x errors (TS5107/TS5101) on `moduleResolution: "node"`
(an alias for the deprecated `node10`) and on `baseUrl`, breaking
`pnpm build`. The entrypoint imports only package specifiers (no
relative imports), so switching to `module`/`moduleResolution`
"NodeNext" is behavior-preserving, and the unused `baseUrl` (no
`paths` map) can be dropped.

Co-authored-by: Cursor <cursoragent@cursor.com>
Adds a root action.yml so other projects' CI can stand up this token
server and consume its /createToken endpoint (mirrors the
livekit/dev-server-action model). The action builds and runs the
server's own code from $GITHUB_ACTION_PATH and waits for readiness,
emitting `token-url` and `log-path` outputs.

Also makes the listen port configurable via the PORT env var so the
action's `port` input is honored and consumers aren't locked to 3000.

Co-authored-by: Cursor <cursoragent@cursor.com>
@alan-george-lk alan-george-lk marked this pull request as ready for review June 22, 2026 22:04

@1egoman 1egoman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Generally makes sense to me, but I'd like @rektdeckard to give the final approval since I want to make sure adding the action in here isn't too antithetical to the goal of this being an example.

Comment thread src/main.ts Outdated
Comment thread README.md Outdated
@alan-george-lk

alan-george-lk commented Jun 23, 2026

Copy link
Copy Markdown
Author

Generally makes sense to me, but I'd like @rektdeckard to give the final approval since I want to make sure adding the action in here isn't too antithetical to the goal of this being an example.

@1egoman Yeah fair play -- happy to move this elsewhere or start a new one from scratch. Thanks

@alan-george-lk

Copy link
Copy Markdown
Author

After pondering/discussion with @1egoman, deciding to move an action to a separate repo as to not pollute the intent of this example.

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.

2 participants