Skip to content

CI rehearsal only (do not merge) - #14

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

michael-moffett wants to merge 10 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 Kotlin test harness to CI pipeline.

No outstanding review finding or new defect was established, though the PR is explicitly marked as a CI rehearsal not to merge.

Summary

The PR adds a Kotlin protocol conformance runner and wires it into harness CI. Since the previous review, it adds runner unit tests to the CI build step and documents the runner’s input/output contract.

Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[CI builds and tests Kotlin runner] --> B[Protocol conformance vectors]
  B --> C[Spawned Kotlin runner]
  C --> D[Kotlin SDK]
  C --> E[Adapter response]
  E --> B
Loading

Reviews (5) · 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 6 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
…lDist test) and Main.kt gains its 3-line header comment
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