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
…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 (pathbase#225).
|
🔍 Preview deployed: https://40a73a38.toolpath.pages.dev |
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
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.
Reported by Pathbase: empathic/pathbase#225 — consumers reading
structural.modelonconversation.appendalways get null, because the model is encoded only in the actor string (agent:<model>) andmeta.actors.Change
modelonconversation.append(toolpath-convo::derive): the structural payload carries the model that produced the turn whenever the source session records one; the key is omitted otherwise — no"unknown"sentinel in payloads. Single emitter, so all seven providers (claude, gemini, codex, pi, cursor, opencode, copilot) gain it at once. Actor strings (includingagent:unknown) andmeta.actorsare unchanged.extractnow prefers the payloadmodeland falls back to parsing theagent:<model>actor for pre-v1.2.0 documents.modelproperty onconversationAppend(only semantic delta from v1.1.0), registered inBUNDLED_KINDS;PATH_KIND_AGENT_CODING_SESSIONnow stamps the v1.2.0 URI with the previous URI preserved as..._V1_1_0; site spec page + indices updated. Consumers that match the kind by slug/major (pathbase does) 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.The companion read-side change (model resolution with actor-def and actor-suffix fallbacks, which also fixes already-uploaded documents retroactively) is empathic/pathbase#304.
Testing
Full quality gate green:
cargo test --workspace(1786 tests) with-D warnings, clippy (workspace + all-targets), fmt, rustdoc-D warnings,path p validate examples/*.json, site build (12 pages, v1.2.0 schema materialized), shimcargo check. New tests pin: payloadmodelpresent when the session records one, absent otherwise (actor staysagent:unknown), payload preferred over actor parse on extract, v1.2.0 bundled/newest-resolution alongside exact-version pinning of v1.0.0/v1.1.0.Need help on this PR? Tag
/codesmithwith what you need. Autofix is disabled.