From 29a88b350fb31fa69a558592964089a2ea140322 Mon Sep 17 00:00:00 2001
From: philluiz2323 <philluiz2323@gmail.com>
Date: Sat, 25 Jul 2026 18:55:22 -0700
Subject: [PATCH] test(miner-ui): cover bare-integer portfolio-queue identifier
 parsing

Add direct resolvePortfolioQueueChatAction tests for release/requeue with a trailing bare integer, plus a negative case where a digit glued to a letter (v2) is not treated as an identifier.

Closes #8671
---
 .../lib/chat-portfolio-queue-resolve.test.ts  | 37 +++++++++++++++++++
 1 file changed, 37 insertions(+)
 create mode 100644 apps/loopover-miner-ui/src/lib/chat-portfolio-queue-resolve.test.ts

diff --git a/apps/loopover-miner-ui/src/lib/chat-portfolio-queue-resolve.test.ts b/apps/loopover-miner-ui/src/lib/chat-portfolio-queue-resolve.test.ts
new file mode 100644
index 0000000000..ebebb99716
--- /dev/null
+++ b/apps/loopover-miner-ui/src/lib/chat-portfolio-queue-resolve.test.ts
@@ -0,0 +1,37 @@
+import { describe, expect, it } from "vitest";
+
+import {
+  PORTFOLIO_QUEUE_CHAT_RELEASE_ACTION,
+  PORTFOLIO_QUEUE_CHAT_REQUEUE_ACTION,
+  resolvePortfolioQueueChatAction,
+} from "./chat-portfolio-queue-resolve";
+
+// (#8671) Bare positive integer after the repo (IDENTIFIER_RE's third arm) was never covered —
+// existing chat-portfolio-queue-actions cases only used `#7` / `issue:12`.
+
+describe("resolvePortfolioQueueChatAction bare-integer identifier (#8671)", () => {
+  it('resolves "release acme/widgets 12" to identifier issue:12', () => {
+    expect(resolvePortfolioQueueChatAction("release acme/widgets 12")).toEqual({
+      ok: true,
+      action: PORTFOLIO_QUEUE_CHAT_RELEASE_ACTION,
+      target: { repoFullName: "acme/widgets", identifier: "issue:12" },
+    });
+  });
+
+  it('resolves "requeue acme/widgets 12" to identifier issue:12', () => {
+    expect(resolvePortfolioQueueChatAction("requeue acme/widgets 12")).toEqual({
+      ok: true,
+      action: PORTFOLIO_QUEUE_CHAT_REQUEUE_ACTION,
+      target: { repoFullName: "acme/widgets", identifier: "issue:12" },
+    });
+  });
+
+  it("does not treat a digit glued to a letter (e.g. v2) as a bare-integer identifier", () => {
+    // Lookbehind/lookahead require the digit not be adjacent to alphanumerics — "v2" must not become issue:2.
+    expect(resolvePortfolioQueueChatAction("release acme/widgets for sprint v2")).toEqual({
+      ok: true,
+      action: PORTFOLIO_QUEUE_CHAT_RELEASE_ACTION,
+      target: { repoFullName: "acme/widgets" },
+    });
+  });
+});