feat: support cross-platform SDK paths - #25
Conversation
This comment has been minimized.
This comment has been minimized.
|
Ready to review this PR? Stage has broken it down into 9 individual chapters for you: Chapters generated by Stage for commit 1c134bd on Jun 16, 2026 10:01am UTC. |
@agent-trail/adapter-kit
@agent-trail/adapters
@agent-trail/catalog
@agent-trail/core
@agent-trail/redact
@agent-trail/render-model
@agent-trail/schema
@agent-trail/sessions
@agent-trail/source-schemas
@agent-trail/store
@agent-trail/types
commit: |
This comment has been minimized.
This comment has been minimized.
|
PR standard violation identified due to the missing Linear issue link and verification commands in the description. Populate the 'Related Issue' and 'Verification' sections as required by .github/PULL_REQUEST_TEMPLATE.md. Kody rule violation: Follow Pull Request Standards |
|
Review loop complete. Addressed findings:
Verification:
No remaining blocking review findings. |
Kody Review CompleteGreat news! 🎉 Keep up the excellent work! 🚀 Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
|
PR standard violation in CLAUDE.md results from the missing Linear issue link. The rule 'Enforce pre-commit checks and PR standards' requires this reference regardless of the maintainer-directed origin. |
|
Pull Request Standards violation occurs because the PR lacks the mandatory Linear issue link required by CLAUDE.md. Compliance requires an associated Linear issue even for maintainer-directed changes. Kody rule violation: Follow Pull Request Standards |
Pull Request
Summary
Related Issue
Public Impact
Impact description:
@agent-trail/storenow supportsresolveStoreRoot({ storeRoot?, env?, platform? })while preservingresolveStoreRoot(string).@agent-trail/adaptersexposes adapter options for custom source paths.@agent-trail/sessionsforwards the same default adapter option types.Verification
rtk bun test packages/store/src/tests/paths.test.tsrtk bun test packages/adapters/src/codex/tests/index.test.ts packages/adapters/src/pi/tests/index.test.tsrtk bun run build:declarationsrtk bun test packages/store/src/tests/paths.test.ts packages/adapters/src/codex/tests/index.test.ts packages/adapters/src/opencode/tests/index.test.ts packages/adapters/src/claude-code/tests/index.test.ts packages/adapters/src/pi/tests/index.test.ts packages/sessions/src/tests/sessions.test.tsrtk mise run check:actionsrtk mise x -- bun run check:changesetsCI=true rtk mise run checkReviewer Notes
CI=truehit opt-in real-session smoke-test timeouts from local environment. CI-mode full check passes and hard-skips those real-session smoke tests as designed.