Repository navigation
Chat composer UX + assistant docs/skill polish - #480
Merged
Merged
Conversation
- Thread "Modified nodes" summary: use read-only Controls.Label instead of Controls.Text (an editable TextFieldControl) for the version transition, header, and new/deleted markers — no more editable input boxes on read-only info. - Chat composer pickers (harness/model/agent): pre-select the currently-active value instead of always highlighting item 0, and close an open picker on Esc from the composer (previously Esc had no effect once focus returned to Monaco). - Docs: prefer `patch` over `update` in ToolsReference + Assistant — Update is now documented as import/restore-only (wholesale overwrite); Patch is the default. - Teach Assistant a UI minimum (markdown in reply / Markdown control / interactive via links; delegate complex UIs to the coder). - Rename the /create-space skill to /space (file + name + references + test list). - AGENTS.md: harden the worktree rule — NEVER work on the primary checkout; it stays parked on main, untouched; every session creates its own worktree. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Contributor
Contributor
Contributor
Contributor
Contributor
Contributor
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.
Summary
A batch of small chat/assistant UX and agent-docs fixes.
Chat composer
v1 → v4), the header, and thenew/deletedmarkers withControls.Text— which is an editableTextFieldControl— so an empty input box appeared on read-only info. Switched those four toControls.Label. Badges, Diff link, and Undo button are unchanged.SelectionId.IdOf, so it works whether the stored value is a bare id or a full node path), falling back to the first row when nothing is selected.OnInputKeyDownonly cancelled the running round; once focus returned to the Monaco editor, Esc did nothing to an open picker. It now dismisses the picker first.Assistant docs & skills
patchoverupdatein ToolsReference + Assistant:Updateis documented as import/restore-only (wholesale overwrite that drops omitted fields);Patchis the default for edits./create-spaceskill to/space(file, frontmatter name, all references, and the two skill-list test assertions).Repo hygiene
main, untouched; every session creates its own worktree.Verification
-warnaserrorbuild ofMeshWeaver.Blazor.Portal(covers ThreadChatView + MeshWeaver.AI + embedded content): green (0/0).SkillNodeTypeTest+SkillHarnessImportSourceTest: 12/12 pass (skill-list assertions updated for the rename).🤖 Generated with Claude Code