Repository navigation
chore: public-repo polish (community files, as-built docs) - #29
Merged
Merged
Conversation
PaulAsjes
approved these changes
Oct 26, 2026
kraenhansen
enabled auto-merge (squash)
October 26, 2026 10:28
kraenhansen
disabled auto-merge
October 26, 2026 10:42
Issue forms, PR template, CODEOWNERS and SECURITY.md (under .github/ so Unity doesn't import them), plus Support and License sections in the README. Templates follow elevenlabs-python's issue forms, adapted for Unity versions, build targets and logs. Closes #20 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
CHANGELOG 0.1.0 lists the API that actually exists, and folds in the Unity 6.3 minimum (never released at 6000.0). ARCHITECTURE, CONTRIBUTING and COMPATIBILITY describe the shipped native transport, asmdef layout, test layers and builds, and limit platform claims to what's tested. CONTRIBUTING gains a "Submitting changes" section. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The missing-session-factory error named the native transport as a future v0.2 addition, and OutputAudioSource's docs referenced a nonexistent EndSessionAsync. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
GitHub withholds repo secrets from fork PRs, so game-ci's Unity activation fails there. Skipping the Edit Mode and WebGL build jobs (the integration job follows via `needs`) keeps outside contributors' PRs from showing red checks; only the lint lane is a required check. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
GitHub withholds secrets from fork-triggered pull_request runs, even after "Approve and run", so the Unity lanes can't activate there. Same-repo PRs and pushes keep running unattended via pull_request. Fork PRs run via pull_request_target, pinned to the PR head SHA, behind the fork-ci environment's required reviewers. Runs inside forks are skipped. Fork runs execute untrusted editor code with license secrets, so they also don't persist checkout credentials or save caches (caches written under pull_request_target land in main's scope). Concurrency groups include the event and PR number so fork runs don't cancel main's. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Removed caution note about early-stage SDK from README.
kraenhansen
force-pushed
the
chore/public-repo-polish
branch
from
October 26, 2026 10:42
71139d4 to
d9949c3
Compare
kraenhansen
enabled auto-merge (squash)
October 26, 2026 10:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Now that the repo is public, outside contributors need issue/PR templates, a security policy and docs that match the shipped SDK. Closes #20 and covers the docs/metadata part of #12 (new samples stay in #26, landing media in #19).
Changes
CODEOWNERSandSECURITY.mdunder.github/, plus Support and License sections in the README.ARCHITECTURE.md,CONTRIBUTING.md(adds "Submitting changes") andCOMPATIBILITY.mdto the as-built native transport, asmdef layout, test layers and builds, and limit platform claims to what's tested.EndSessionAsynccref.fork-cienvironment (forkpull_requestruns never get theUNITY_*secrets); same-repo PRs and pushes keep running unattended.Verification
dotnet csharpier check .Bridge~checks (format:check,lint,typecheck,test,verify:primitives,verify:connection)Codegen~checks (verify:protocol-dtos,round-trip)pnpm --dir TestProject run test)IntegrationTests~)Checklist
CHANGELOG.mdupdated under[Unreleased]for user-facing changesAdditional context intended for coding agents and bots
Design notes
SECURITY.mdlives in.github/(GitHub reads it there) so Unity doesn't import it into the package or need a.meta. It points at GitHub private vulnerability reporting (already enabled) and the ElevenLabs VDP fromelevenlabs.io/.well-known/security.txt.elevenlabs/elevenlabs-python(the only DX SDK repo with templates), adapted for Unity version, build target and Editor/Player/browser logs. No sibling DX repo has a PR template, CODEOWNERS or a usable security policy to copy.CODEOWNERSis* @kraenhansen.[Unreleased]line, since 0.1.0 has never been tagged; that's why the "CHANGELOG updated under [Unreleased]" box is unticked.pull_request; fork PRs usepull_request_targetwithenvironment: fork-ci(required reviewer @kraenhansen, already created), checking outpull_request.head.shawithpersist-credentials: false. Runs inside forks are skipped. Fork runs restore but never save caches, sincepull_request_targetcaches land inmain's scope. Concurrency groups include event + PR number, becausegithub.refismainunderpull_request_target.pull_request_targetuses the workflow frommain, so the fork path takes effect after merge.HostBuild.BuildStandalone) and WebGL are built or tested, so iOS/Android/XR are listed as untested; native WebRTC is documented as unsupported (NotSupportedException).Known follow-ups (not in this PR)
.github/workflows/lint.yml's header comment references a nonexistentDocs~/plans/ci.mdand says the Unity lanes are blocked on licensing.needs reproduction,has reproduction,needs information,tracking,confirmed,upstream,stale) used by sibling SDK repos.Test plan and verification
unity test TestProject --mode EditMode -- -nographics: 460/460 passed.dotnet csharpier check Runtime: clean..github/andREADME.md(parses the issue-form YAML): clean.Conversation/ConversationOptions; doc claims cross-checked against code by a read-only audit pass.discord.com/invite/elevenlabs, ElevenAgents docs,elevenlabs.io/security/report) return 200.🤖 Generated with Claude Code