Parse colon-less picker ids without truncating the provider

slice(0, indexOf(":")) drops the last character when the id has no colon, mis-attributing the row. split(":")[0] keeps the full id.

Author: TheGreatAxios

src/tui/model-catalog.test.ts

@@ -244,4 +244,14 @@ describe("describeModelCatalogOption", () => {
     expect(description?.impact).not.toMatch(/pricing unknown/i);
     expect(description?.impact).toMatch(/subscription/);
   });
+
+  test("colon-less ids keep the full provider instead of dropping the last character", () => {
+    // slice(0, indexOf(":")) truncates a colon-less id (indexOf is -1), so
+    // "codex/" became "code" and missed subscription billing.
+    const description = describeModelCatalogOption(
+      { id: "codex/", label: "default * [Codex default]" },
+      { pricing: null },
+    );
+    expect(description?.impact).toMatch(/ChatGPT subscription/);
+  });
 });

src/tui/model-catalog.ts

@@ -324,7 +324,9 @@ export function describeModelCatalogOption(
     what: whatLine(model),
     impact: pricingImpact(
       pricing,
-      option.id.slice(0, option.id.indexOf(":")),
+      // Exact provider parse: slice(0, indexOf(":")) drops the last
+      // character of a colon-less id (indexOf returns -1).
+      option.id.split(":")[0] ?? option.id,
       model,
     ),
     tone: "plain",