fix(chat): drop /·$ prefix from command and skill badges in message transcript

Command and skill badges in the sent-message transcript now display the bare
name without the leading /·$ prefix, matching the composer's inline badges. The
wire invocation token (/command, $skill) is unchanged, so agents still receive
the literal text.

Author: xintaofei

src/components/chat/composer/reference-uri.test.ts

@@ -114,21 +114,25 @@ describe("parseCodegReferenceUri", () => {
     })
   })
 
-  it("parses a skill uri, keeping the literal `/`·`$` token as the label", () => {
+  it("parses a skill uri, stripping the leading `/`·`$` from the label", () => {
     expect(
       parseCodegReferenceUri("codeg://skill/review", "/review")
     ).toMatchObject({
       refType: "skill",
       id: "review",
-      label: "/review",
+      label: "review",
       uri: "codeg://skill/review",
       meta: null,
     })
+    // The `$` prefix ($skill / Codex expert) is stripped the same way.
+    expect(
+      parseCodegReferenceUri("codeg://skill/deploy", "$deploy")?.label
+    ).toBe("deploy")
   })
 
-  it("falls back to a /-prefixed id for an empty skill label", () => {
+  it("falls back to the bare id for an empty skill label", () => {
     expect(parseCodegReferenceUri("codeg://skill/deploy", "")?.label).toBe(
-      "/deploy"
+      "deploy"
     )
   })
 

src/components/chat/composer/reference-uri.ts

@@ -11,7 +11,8 @@ const AGENT_URI = /^codeg:\/\/agent\/(.+)$/i
 const SESSION_URI = /^codeg:\/\/session\/(.+)$/i
 const COMMIT_URI = /^codeg:\/\/commit\/.*@(.+)$/i
 // command / skill / expert tokens, surfaced as badges in transcript user messages
-// (message/user-message-segments.ts). The label carries the literal `/`·`$` prefix.
+// (message/user-message-segments.ts). The label arrives with the literal `/`·`$`
+// prefix, which the skill branch strips so the badge reads the bare name.
 const SKILL_URI = /^codeg:\/\/skill\/(.+)$/i
 
 // A path-less attached file (local-desktop paste/drop of inline bytes — an
@@ -118,10 +119,13 @@ export function parseCodegReferenceUri(
     }
     return {
       refType: "skill",
-      // The link text keeps the literal token (`/build` / `$deploy`); fall back
-      // to a `/`-prefixed id only if it was somehow empty.
+      // The link text carries the literal invocation token (`/build` / `$deploy`);
+      // strip a single leading `/`·`$` so the badge reads `build` — matching the
+      // composer's live-inserted command/skill badge (whose label is the bare,
+      // prefix-less name), the way the agent branch above strips a leading `@`.
+      // Fall back to the bare id when the label is empty.
       id,
-      label: label || `/${id}`,
+      label: (label || id).replace(/^[/$]/, "") || id,
       uri,
       meta: null,
     }

src/components/message/plain-text-with-badges.test.tsx

@@ -54,7 +54,9 @@ describe("PlainTextWithBadges", () => {
       <PlainTextWithBadges text="run /review please" />
     )
     expect(badge(cmd, "skill")).not.toBeNull()
-    expect(cmd.textContent).toContain("/review")
+    // The badge shows the bare name (no `/` prefix), matching the composer.
+    expect(cmd.textContent).toContain("review")
+    expect(cmd.textContent).not.toContain("/review")
 
     const { container: path } = render(
       <PlainTextWithBadges text="see /usr/bin for it" />

src/components/message/user-message-segments.test.ts

@@ -95,19 +95,20 @@ describe("parseUserMessageSegments", () => {
   })
 
   describe("bare invocation tokens → skill badges", () => {
-    it("badges a /command token keeping its prefix", () => {
+    it("badges a /command token, dropping its prefix from the label", () => {
       const segments = parseUserMessageSegments("run /review please")
       expect(segments[0]).toEqual({ kind: "text", text: "run " })
       const skill = segments[1] as { kind: "reference"; attrs: ReferenceAttrs }
       expect(skill.attrs.refType).toBe("skill")
-      expect(skill.attrs.label).toBe("/review")
+      // Badge label matches the composer's inline badge: the bare name, no `/`.
+      expect(skill.attrs.label).toBe("review")
       expect(segments[2]).toEqual({ kind: "text", text: " please" })
     })
 
-    it("badges a $skill token", () => {
+    it("badges a $skill token, dropping its prefix from the label", () => {
       const attrs = onlyReference("$deploy now")
       expect(attrs.refType).toBe("skill")
-      expect(attrs.label).toBe("$deploy")
+      expect(attrs.label).toBe("deploy")
     })
 
     it("does NOT badge a file-ish path", () => {

src/components/message/user-message-segments.ts

@@ -33,9 +33,10 @@ function unwrapDestination(destination: string): string {
 
 /**
  * Split a plain-prose run into literal text and bare `/slug`·`$slug` skill
- * badges. Mirrors `rehype-command-badges` (same {@link INVOCATION_TOKEN_RE}) so a
- * sent invocation token renders identically to how the assistant-side transcript
- * used to badge it — the skill badge label keeps the literal `/`·`$` prefix.
+ * badges (same {@link INVOCATION_TOKEN_RE} the composer's `/`·`$` triggers use).
+ * The badge label drops the literal `/`·`$` prefix (the parser strips it) so a
+ * sent invocation token renders identically to the composer's inline badge,
+ * which shows the bare command/skill name.
  */
 function pushProseSegments(value: string, out: UserMessageSegment[]): void {
   INVOCATION_TOKEN_RE.lastIndex = 0
@@ -48,8 +49,9 @@ function pushProseSegments(value: string, out: UserMessageSegment[]): void {
       out.push({ kind: "text", text: value.slice(lastIndex, tokenStart) })
     }
     const slug = token.slice(1)
-    // Resolve through the same parser the transcript uses for `codeg://skill/…`
-    // so the badge label keeps the literal token (`/build`, `$deploy`).
+    // Resolve through the shared reference parser, which strips the leading
+    // `/`·`$` so the badge label is the bare slug (`build`, `deploy`) — matching
+    // the composer's inline command/skill badge.
     const attrs = parseCodegReferenceUri(
       `codeg://skill/${encodeURIComponent(slug)}`,
       token