This repository was archived by the owner on Mar 26, 2026. It is now read-only.
fix(ui): TASK-139/140 — task panel width, markdown newlines, PR link#298
Merged
fix(ui): TASK-139/140 — task panel width, markdown newlines, PR link#298
Conversation
…-setup.sh Aligns scripts with CLAUDE.md docs (PR #290). MCP server name changed from boss-dev to odis-dev in the generated JSON config, allowed tools list, and output messages. BOSS_API_TOKEN/BOSS_MCP_URL env vars kept as fallbacks for backward compatibility (prefer ODIS_API_TOKEN/ODIS_MCP_URL going forward). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…our Conversations Root cause: both 'boss' (legacy) and 'operator' agents have agent_type='human' in the DB. operatorName computed returned whichever was found first via Object.entries, which is insertion-order. In spaces where 'boss' was created first, operatorName='boss'. Consequences when operatorName='boss' but messages use 'operator': - Conversations with 'operator' participant went to Agent Conversations section - composeRecipient was null for those conversations (no compose box shown) - Unread acks used wrong agent name Fix: collect ALL human agents into humanAgentNames Set. Use it for: - isOperatorConversation: any participant that is a human agent - composeRecipient: any participant that is NOT a human agent - unreadCount / ackOperatorMessages: any human-agent recipient - operatorName: prefer 'operator' (canonical) over legacy 'boss' Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tion threads When agents use different capitalizations (e.g. "CEO" vs "ceo"), messages were stored with the raw sender name, creating separate conversation threads in the UI for what should be a single conversation. Both handleAgentMessage (HTTP) and send_message (MCP tool) now call resolveAgentName on the sender after the space is loaded, mapping to the canonical stored name if the sender is a known agent. Unknown senders keep their original name unchanged. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Address remaining boss feedback after PR #134 (CLOSED): - Remove [?BOSS] tag syntax from docs/AGENT_PROTOCOL.md questions field example and description; replace with plain language - Replace wall-clock status cadence with milestone-based language in AGENT_PROTOCOL.md (Pattern A work loop + non-tmux checklist) - messaging-protocol.md: remove 'manager can explicitly forbid' qualifier from peer-to-peer section; peer messaging is always allowed, no authorization needed - messaging-protocol.md: reframe escalation to walk up the chain rather than hardcoding a 'boss' message target with 30-minute wall-time rule - organizational-model.md: remove 'manager can restrict' qualifier from information-flow principle; direct messaging is always allowed - ignition-prompts.md: remove 'boss agent channel for boss-level decisions' from hierarchy section; add blocked task status to blocker flow - ignition-prompts.md: soften org chart note — position set by spawning agent, not a warning about org instability - team-formation.md: remove time-based threshold (~30 min); replace with action-based criteria (multiple focused actions = non-trivial); strengthen the default-to-team recommendation Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
TASK-139: - Widen SheetContent from 480/540px to 560/680px for more readable content - Fix scroll structure: remove outer overflow-y-auto so header stays pinned while body scrolls independently (flex-1 overflow-y-auto on body) - Add leading-relaxed to description for better line-height readability TASK-140: - buildPrUrl now handles github.com/gitlab.com URLs missing https:// prefix - Improved PR number extraction regex handles more formats (PR #42, PR-42) - Fallback display for unresolvable linked_pr changed from muted plain text to a styled code badge with tooltip explaining missing repo_url config Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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
Closes #293
Closes #294
Fixes two issues reported in GitHub issues #293 and #294 for the task right panel.
TASK-139 — Panel width + markdown newlines (#293)
SheetContentfrom480/540pxto560/680px— more room for descriptions, comments, and metadataoverflow-y-autofrom outerSheetContentso the sticky header stays pinned while the body scrolls independently viaflex-1 overflow-y-autoleading-relaxedto description div for better line-height readability with multi-line contentTASK-140 — PR link field not clickable (#294)
buildPrUrlnow handlesgithub.com/...andgitlab.com/...URLs that are missing thehttps://prefixPR #42,PR-42, bare42)linked_pr(norepo_urlon agents) changed from muted plain text to a styled<code>badge with a tooltip explaining what's needed to make it a linkTest plan
leading-relaxedlinked_pr = "#123"and no agentrepo_url— shows styled code badge with tooltip, not just gray textlinked_pr = "https://github.com/..."— still renders as clickable linklinked_pr = "github.com/org/repo/pull/42"(no https prefix) — now renders as clickable linkgo test -race ./internal/coordinator/— passesnpm run build— clean build, no TS errors🤖 Generated with Claude Code