feat(convo): emit model on conversation.append; kind agent-coding-session v1.2.0 - #132
Open
akesling wants to merge 1 commit into
Open
feat(convo): emit model on conversation.append; kind agent-coding-session v1.2.0#132akesling wants to merge 1 commit into
akesling wants to merge 1 commit into
Conversation
|
🔍 Preview deployed: https://e45e1a05.toolpath.pages.dev |
akesling
force-pushed
the
akesling/append-model-field
branch
2 times, most recently
from
August 4, 2026 15:30
03fff1a to
dd7ffb0
Compare
akesling
force-pushed
the
akesling/append-model-field
branch
from
August 6, 2026 17:47
0031dc9 to
2c3eaa4
Compare
…sion v1.2.0 Assistant turns' structural payloads now carry the model that produced them, so documents are self-describing without parsing the actor string. The key is emitted only when the source session records a model; extraction prefers the payload field and falls back to the agent:<model> actor parse for pre-v1.2.0 documents. Actor strings and meta.actors are unchanged. The optional model field is documented in a new agent-coding-session v1.2.0 kind (bundled schema, site spec page); derived documents are stamped with the v1.2.0 URI. Crates bumped per the kind-URI-change policy: every deriver and the CLIs. Reported by Pathbase, whose stats view grouped every token under "(unknown)" for derived documents.
akesling
force-pushed
the
akesling/append-model-field
branch
from
August 6, 2026 18:24
2c3eaa4 to
b192823
Compare
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
1 participant
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.
Stacked on #164 (kind-URI consolidation) — review that first; this targets its branch and will retarget to
mainwhen it merges.Reported by Pathbase: consumers reading
structural.modelonconversation.appendalways get null, because the model is encoded only in the actor string (agent:<model>) andmeta.actors, so its stats view grouped every token under one unknown model.Change
modelonconversation.appendintoolpath-convo::derive: the structural payload carries the model that produced the turn whenever the source records one; the key is omitted otherwise — no placeholder ever lands in a payload. There is a single emitter, so all seven providers (claude, gemini, codex, pi, cursor, opencode, copilot) gain it at once. Actor strings andmeta.actorsare unchanged.extractprefers the payloadmodeland falls back to parsing theagent:<model>actor for pre-v1.2.0 documents.modelproperty onconversationAppend— the only semantic delta from v1.1.0 — registered inBUNDLED_KINDS, withPATH_KIND_AGENT_CODING_SESSIONnow stamping the v1.2.0 URI and the previous URI preserved as..._V1_1_0. Site spec page and indices updated. Consumers that match the kind by slug or major version are unaffected.Cargo.lock,site/_data/crates.json, and thetoolpath-clishim kept in sync. All bumps are breaking-level under 0.x caret semantics, so downstream pins do not auto-flow.Because #164 lands first, minting here touches only the constant, the schema file, its
BUNDLED_KINDSentry, one pin test, the site page, and the changelog — five Rust files instead of ten.Not in this PR
The harness-generated-turn fix (a deriver bug fix with no schema change) is #165, independent of this work. The companion read-side change in Pathbase — model resolution with actor-definition and actor-suffix fallbacks — fixes already-uploaded documents retroactively.
Testing
Full workspace suite, clippy, rustdoc, the examples gate, and the site build green. New tests pin: the payload
modelpresent when the session records one and absent otherwise, payload preferred over the actor parse on extract, and v1.2.0 bundled and resolving as newest alongside exact-version pinning of v1.0.0 and v1.1.0.