Repository navigation
Conversation
…ns this thread's ID T3 has no t3_thread_configuration tool. t3_thread_configure with no threadId reads this thread's configuration, which carries its threadId.
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.
Why
show-me-your-work tells an agent to get this run's thread ID from
t3_thread_configuration. T3 has no tool by that name, so the call fails and the agent has to find the ID another way before it can check its log against the thread.The tool is
t3_thread_configure(apps/server/src/mcp/toolkits/thread/tools.tsin pingdotgg/t3code atd5a50a1). Its description says "OmitthreadIdfor this thread", and with nomodelSelectionit only reads. The configuration it returns includesthreadId.What changed
tools/substitutions.jsonthat writes this sentence now namest3_thread_configure.show-me-your-work/SKILL.mdhas the same one-word change, so it stays a derived file and does not become a fork.The other 15 T3 tools the plugin names all exist in t3code at
d5a50a1.Version
I did not bump
VERSION. #72 already uses 2.2.9 and #78 uses 2.3.0, so a third number would only conflict. This can ship in whichever release merges next. If you want it as a release of its own, tell me and I will add the bump.Checks
bun tools/sync.mjs <component> c742dd6 --dry-runexits 0 forpstack,cursor-team-kitandthermos. With the rule change reverted and only the derived file edited, thepstackdry run exits 1 and reportsshow-me-your-work/SKILL.mdasundeclared.bun tools/generate.mjs --checkreports 17 generated files current and every check passing.GIT_CONFIG_GLOBAL=/dev/null bun test tests/ --timeout 30000: 546 pass, 69 skip, 2 fail. The 2 failures are both in "the fork check", and they fail the same way on unmodifiedmainon this machine (NixOS, Bun from nixpkgs), so this change does not cause them.