fix(composer): only chip skills offered by the current provider

apps/mobile/src/lib/composerContext.test.ts

@@ -159,10 +159,25 @@ describe("mobile composer context", () => {
   it("keeps exact source positions and repeated references alongside existing native tokens", () => {
     const reference = formatComposerContextReference(terminal);
     const text = `Use $playwright and [app.ts](src/app.ts) with ${reference} then ${reference}`;
-    const tokens = composerContextEditorTokens(text, collectComposerInlineTokens(text));
+    const tokens = composerContextEditorTokens(
+      text,
+      collectComposerInlineTokens(text),
+      new Set(["playwright"]),
+    );
     expect(tokens.map((token) => token.type)).toEqual(["skill", "mention", "context", "context"]);
     for (const token of tokens) expect(text.slice(token.start, token.end)).toBe(token.source);
   });
+  it("leaves a $name that is not one of the provider's skills as plain text", () => {
+    const text = "Run $playwright then echo $HOME and $notaskill ";
+    const tokensFor = (skills: ReadonlyArray<string>) =>
+      composerContextEditorTokens(text, collectComposerInlineTokens(text), new Set(skills)).map(
+        (token) => token.source,
+      );
+    expect(tokensFor(["playwright"])).toEqual(["$playwright"]);
+    // The skill list loads after the editor opens, and differs per provider.
+    expect(tokensFor([])).toEqual([]);
+    expect(tokensFor(["playwright", "notaskill"])).toEqual(["$playwright", "$notaskill"]);
+  });
 
   it("removes deleted payloads but keeps the screenshot linked to a remaining annotation", () => {
     const context: OrchestrationMessageContext = {

apps/mobile/src/lib/composerContext.ts

@@ -144,11 +144,18 @@ export function pullRequestComposerContext(
 }
 
 /** Native editors collapse the canonical source range to a single atomic attachment. */
-export function composerContextEditorTokens(text: string, tokens: readonly ComposerInlineToken[]) {
+export function composerContextEditorTokens(
+  text: string,
+  tokens: readonly ComposerInlineToken[],
+  knownSkills: { has(name: string): boolean },
+) {
   const references = collectComposerContextReferences(text);
   return [
     ...tokens.filter(
-      (token) => !references.some((ref) => token.start < ref.end && token.end > ref.start),
+      (token) =>
+        // `$HOME` and other `$name` text that is not one of the provider's skills stays prose.
+        (token.type !== "skill" || knownSkills.has(token.value)) &&
+        !references.some((ref) => token.start < ref.end && token.end > ref.start),
     ),
     ...references.map((ref) => ({
       type: "context" as const,

apps/mobile/src/native/T3ComposerEditor.ios.tsx

@@ -168,7 +168,7 @@ export function ComposerEditor({
     });
     confirmedTokensRef.current = tokens;
     return JSON.stringify(
-      composerContextEditorTokens(props.value, tokens).map((token) => {
+      composerContextEditorTokens(props.value, tokens, skillLabels).map((token) => {
         const record =
           token.type === "context"
             ? props.context?.records.find((record) => record.contextId === token.contextId)

apps/mobile/src/native/T3ComposerEditor.native.tsx

@@ -166,7 +166,7 @@ export function ComposerEditor({
     });
     confirmedTokensRef.current = tokens;
     return JSON.stringify(
-      composerContextEditorTokens(props.value, tokens).map((token) => {
+      composerContextEditorTokens(props.value, tokens, skillLabels).map((token) => {
         const record =
           token.type === "context"
             ? props.context?.records.find((record) => record.contextId === token.contextId)