Skip to content

chore: vendor accordkit SDK in-tree at packages/accordkit#105

Merged
krazyjakee merged 1 commit into
masterfrom
claude/youthful-williamson-fc3074
Jun 10, 2026
Merged

chore: vendor accordkit SDK in-tree at packages/accordkit#105
krazyjakee merged 1 commit into
masterfrom
claude/youthful-williamson-fc3074

Conversation

@krazyjakee

Copy link
Copy Markdown
Contributor

Summary

  • Move the Accord protocol SDK from the GitHub git dependency (DaccordProject/accordkit-dart) into the repo at packages/accordkit, so it can be maintained alongside the client.
  • Switch pubspec.yaml from the git: source to path: packages/accordkit; flutter pub get re-resolved cleanly (reflected in pubspec.lock).
  • Update CLAUDE.md references (networking note, packages/ layout, setup snippet, "when in doubt" pointer) to reflect the in-tree location.

Why

We're consolidating maintenance of the SDK into this repo. The upstream accordkit-dart repo has been archived (read-only); this tree is now canonical.

Notes for reviewer

  • The SDK source was vendored as a plain copy (no .git), matching the existing packages/livekit_client and packages/markdown_viewer pattern — copied at upstream commit 7e05060.
  • Build artifacts (.dart_tool, build, pubspec.lock, IDE dirs) were excluded from the copy.
  • Dependency resolution verified via flutter pub get; a full flutter analyze was not run.

🤖 Generated with Claude Code

Move the Accord protocol SDK from the GitHub git dependency
(DaccordProject/accordkit-dart) into the repo so it can be maintained
alongside the client. Switches pubspec to a path dependency and updates
CLAUDE.md references. The upstream repo has been archived.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@krazyjakee krazyjakee merged commit ad2cb4e into master Jun 10, 2026
5 checks passed
krazyjakee added a commit that referenced this pull request Jun 10, 2026
@krazyjakee krazyjakee deleted the claude/youthful-williamson-fc3074 branch June 28, 2026 23:01
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