Skip to content

CI rehearsal only (do not merge) - #14

Open
michael-moffett wants to merge 8 commits into
mainfrom
pay-kit-89-m1-kotlin-runner-a1-next
Open

michael-moffett wants to merge 8 commits into
mainfrom
pay-kit-89-m1-kotlin-runner-a1-next

Conversation

@michael-moffett

Copy link
Copy Markdown
Member

Fork CI only. Do not merge.

michael-moffett and others added 2 commits October 3, 2026 06:46
…matrix

A Gradle stdin/stdout runner over the Kotlin SDK's protocol functions, plus `harness/protocol-runners/kotlin.json`, so the spawned-runner block drives Kotlin.

No protocol-layer vector reaches Kotlin today. Milestone 1 of our harness proposal puts Kotlin in the divergence matrix.
@greptile-apps

greptile-apps Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[High risk] Adds CI test infrastructure for a new language runtime.

No blocking issue was identified in the changes since the previous review.

Summary

The PR adds a Kotlin mpp-protocol runner and CI conformance step, then extends the spawned-runner tests to cover Kotlin’s supported credential formatter and declared SDK gaps.

  • The latest changes add receipt-format divergence coverage and use the TypeScript parser to compare Kotlin’s formatted credential with the canonical result.
  • No new actionable issue was identified.

Reviews (4) · Last reviewed commit: "greptile.json: review on request only"

Comment thread harness/test/protocol-conformance.test.ts Outdated
Comment thread harness/test/protocol-conformance.test.ts Outdated
michael-moffett and others added 2 commits October 3, 2026 06:52
…-run steps in the kotlin harness job, and asserted Kotlin divergences in the spawned-runner matrix
michael-moffett and others added 4 commits October 3, 2026 13:46
…cratch (--rerun-tasks); 5/5 MainTest cases pass, the vitest suite is still unrun
…l as a must-pass case (TS reference re-parses Kotlin's header) and lists receipt.format as a known unsupported divergence
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