team-toon-tack 2.0.3 → 2.1.0

package/README.md

@@ -10,7 +10,8 @@ Optimized Linear workflow for Claude Code — saves significant tokens compared
 - **Smart Task Selection** — Auto-pick highest priority unassigned work with `/work-on next`
 - **Multi-team Support** — Sync and filter issues across multiple teams
 - **Flexible Sync Modes** — Choose between remote (immediate Linear sync) or local (offline-first, sync later with `--update`)
-- **QA/PM Team Support** — Auto-update parent issues in QA/PM team to "Testing" when completing dev tasks
+- **Completion Modes** — Four modes for task completion: simple, strict review, upstream strict, upstream not strict
+- **QA Team Support** — Auto-update parent issues in QA team to "Testing" when completing dev tasks
 - **Attachment Download** — Auto-download Linear images and files to local `.ttt/output/` for AI vision analysis
 - **Blocked Status** — Set tasks as blocked when waiting on external dependencies
 - **Auto Command Setup** — `ttt init` can install Claude Code commands with custom prefix
@@ -30,10 +31,22 @@ ttt init
 ```
 
 During init, you'll configure:
+- **Dev team**: Your development team (single selection)
+- **Dev testing status**: Testing/review status for your dev team (optional)
+- **QA team(s)**: For cross-team parent issue updates, each with its own testing status (optional)
+- **Completion mode**: How task completion is handled (see below)
 - **Status source**: `remote` (update Linear immediately) or `local` (work offline, sync with `ttt sync --update`)
-- **QA/PM team**: For cross-team parent issue updates (parent must be set in Linear)
 - **Claude Code commands**: Auto-install with optional prefix (e.g., `/ttt:work-on`)
 
+### Completion Modes
+
+| Mode | Behavior |
+|------|----------|
+| `simple` | Mark task as Done + parent as Done. Default when no QA team configured. |
+| `strict_review` | Mark task to dev testing + parent to QA testing. |
+| `upstream_strict` | Mark task as Done + parent to Testing. Falls back to dev testing if no parent. Default when QA team configured. |
+| `upstream_not_strict` | Mark task as Done + parent to Testing. No fallback if no parent. |
+
 ### 2. Daily Workflow
 
 In Claude Code:

package/README.zh-TW.md

@@ -10,7 +10,8 @@
 - **智慧任務挑選** — `/work-on next` 自動選擇最高優先級的未指派工作
 - **多團隊支援** — 跨多個團隊同步與過濾 issue
 - **彈性同步模式** — 選擇 remote（即時同步 Linear）或 local（離線優先，稍後用 `--update` 同步）
-- **QA/PM 團隊支援** — 完成開發任務時自動將 QA/PM 團隊的 parent issue 更新為「Testing」
+- **完成模式** — 四種任務完成模式：簡單、嚴格審查、上下游嚴格、上下游非嚴格
+- **QA 團隊支援** — 完成開發任務時自動將 QA 團隊的 parent issue 更新為「Testing」
 - **附件下載** — 自動下載 Linear 圖片和檔案到本地 `.ttt/output/`，供 AI 視覺分析
 - **阻塞狀態** — 等待外部依賴時可設定任務為 blocked
 - **自動安裝指令** — `ttt init` 可自動安裝 Claude Code commands，支援自訂前綴
@@ -30,10 +31,22 @@ ttt init
 ```
 
 初始化時會設定：
+- **開發團隊**：你的開發團隊（單選）
+- **開發團隊測試狀態**：開發團隊的 testing/review 狀態（可選）
+- **QA 團隊**：跨團隊 parent issue 更新，各自設定 testing 狀態（可選）
+- **完成模式**：任務完成時的處理方式（見下方說明）
 - **狀態來源**：`remote`（即時更新 Linear）或 `local`（離線工作，用 `ttt sync --update` 同步）
-- **QA/PM 團隊**：跨團隊 parent issue 更新（需在 Linear 設定 parent）
 - **Claude Code commands**：自動安裝，可選前綴（如 `/ttt:work-on`）
 
+### 完成模式
+
+| 模式 | 行為 |
+|------|------|
+| `simple` | 任務標記為 Done，parent 也標記為 Done。未設定 QA 團隊時的預設值。 |
+| `strict_review` | 任務標記到開發團隊的 testing 狀態，parent 標記到 QA 團隊的 testing 狀態。 |
+| `upstream_strict` | 任務標記為 Done，parent 移動到 Testing。若無 parent，fallback 到開發團隊的 testing 狀態。設定 QA 團隊時的預設值。 |
+| `upstream_not_strict` | 任務標記為 Done，parent 移動到 Testing。若無 parent 不做 fallback。 |
+
 ### 2. 每日工作流
 
 在 Claude Code 中：

package/dist/scripts/config/teams.js

@@ -1,5 +1,6 @@
 import prompts from "prompts";
 import { getLinearClient, saveLocalConfig, } from "../utils.js";
+import { getDefaultStatusTransitions } from "../lib/config-builder.js";
 export async function configureTeams(_config, localConfig) {
     console.log("👥 Configure Teams\n");
     const client = getLinearClient();
@@ -9,51 +10,182 @@ export async function configureTeams(_config, localConfig) {
         console.error("No teams found.");
         process.exit(1);
     }
-    // Multi-select teams
-    const teamsResponse = await prompts({
-        type: "multiselect",
-        name: "teamKeys",
-        message: "Select teams to sync (space to select):",
+    // Build team key mapping
+    const teamKeyMap = new Map(); // id -> key
+    for (const team of teams) {
+        const key = team.name.toLowerCase().replace(/[^a-z0-9]/g, "_");
+        teamKeyMap.set(team.id, key);
+    }
+    // Fetch workflow states for all teams
+    const teamStatesMap = new Map();
+    for (const team of teams) {
+        try {
+            const statesData = await client.workflowStates({
+                filter: { team: { id: { eq: team.id } } },
+            });
+            teamStatesMap.set(team.id, statesData.nodes);
+        }
+        catch {
+            teamStatesMap.set(team.id, []);
+        }
+    }
+    // 1. Select dev team (single)
+    console.log("\n👨‍💻 Dev Team:");
+    const devTeamResponse = await prompts({
+        type: "select",
+        name: "teamId",
+        message: "Select your dev team:",
         choices: teams.map((t) => {
-            const key = t.name.toLowerCase().replace(/[^a-z0-9]/g, "_");
-            const currentTeams = localConfig.teams || [localConfig.team];
+            const key = teamKeyMap.get(t.id) || "";
             return {
                 title: t.name,
-                value: key,
-                selected: currentTeams.includes(key),
+                value: t.id,
+                selected: key === localConfig.team,
             };
         }),
-        min: 1,
     });
-    const selectedTeamKeys = teamsResponse.teamKeys || [localConfig.team];
-    // If multiple teams, ask for primary
-    let primaryTeam = localConfig.team;
-    if (selectedTeamKeys.length > 1) {
-        const primaryResponse = await prompts({
-            type: "select",
-            name: "primary",
-            message: "Select primary team:",
-            choices: selectedTeamKeys.map((key) => ({
-                title: key,
-                value: key,
+    const devTeamId = devTeamResponse.teamId;
+    const devTeamKey = teamKeyMap.get(devTeamId) || localConfig.team;
+    const devTeam = teams.find((t) => t.id === devTeamId);
+    // 2. Select dev team testing status
+    const devStates = teamStatesMap.get(devTeamId) || [];
+    const devDefaults = getDefaultStatusTransitions(devStates);
+    console.log("\n🔍 Dev Team Testing/Review Status:");
+    const devTestingResponse = await prompts({
+        type: "select",
+        name: "testingStatus",
+        message: "Select testing/review status for dev team:",
+        choices: [
+            { title: "(Skip - no testing status)", value: undefined },
+            ...devStates.map((s) => ({
+                title: `${s.name} (${s.type})`,
+                value: s.name,
             })),
-            initial: selectedTeamKeys.indexOf(localConfig.team),
+        ],
+        initial: localConfig.dev_testing_status
+            ? devStates.findIndex((s) => s.name === localConfig.dev_testing_status) +
+                1
+            : devDefaults.testing
+                ? devStates.findIndex((s) => s.name === devDefaults.testing) + 1
+                : 0,
+    });
+    const devTestingStatus = devTestingResponse.testingStatus;
+    // 3. Select QA/PM teams (multiple)
+    const otherTeams = teams.filter((t) => t.id !== devTeamId);
+    const qaPmTeams = [];
+    if (otherTeams.length > 0) {
+        console.log("\n🔗 QA/PM Teams:");
+        const qaPmResponse = await prompts({
+            type: "multiselect",
+            name: "teamIds",
+            message: "Select QA/PM teams for cross-team parent updates (space to select):",
+            choices: otherTeams.map((t) => {
+                const key = teamKeyMap.get(t.id) || "";
+                const currentQaPm = localConfig.qa_pm_teams || [];
+                return {
+                    title: t.name,
+                    value: t.id,
+                    selected: currentQaPm.some((qp) => qp.team === key),
+                };
+            }),
+            hint: "- Press space to select, enter to confirm. Leave empty to skip.",
         });
-        primaryTeam = primaryResponse.primary || selectedTeamKeys[0];
+        const selectedQaPmIds = qaPmResponse.teamIds || [];
+        // 4. For each QA/PM team, select testing status
+        for (const teamId of selectedQaPmIds) {
+            const team = teams.find((t) => t.id === teamId);
+            if (!team)
+                continue;
+            const teamKey = teamKeyMap.get(teamId) || "";
+            const teamStates = teamStatesMap.get(teamId) || [];
+            const defaults = getDefaultStatusTransitions(teamStates);
+            // Find existing config for this team
+            const existingConfig = localConfig.qa_pm_teams?.find((qp) => qp.team === teamKey);
+            const stateChoices = teamStates.map((s) => ({
+                title: `${s.name} (${s.type})`,
+                value: s.name,
+            }));
+            const statusResponse = await prompts({
+                type: "select",
+                name: "testingStatus",
+                message: `Select testing status for ${team.name}:`,
+                choices: [
+                    { title: "(Skip this team)", value: undefined },
+                    ...stateChoices,
+                ],
+                initial: existingConfig?.testing_status
+                    ? stateChoices.findIndex((c) => c.value === existingConfig.testing_status) + 1
+                    : defaults.testing
+                        ? stateChoices.findIndex((c) => c.value === defaults.testing) + 1
+                        : 0,
+            });
+            if (statusResponse.testingStatus) {
+                qaPmTeams.push({
+                    team: teamKey,
+                    testing_status: statusResponse.testingStatus,
+                });
+            }
+        }
     }
-    else {
-        primaryTeam = selectedTeamKeys[0];
-    }
-    localConfig.team = primaryTeam;
-    localConfig.teams =
-        selectedTeamKeys.length > 1 ? selectedTeamKeys : undefined;
+    // 5. Select completion mode
+    console.log("\n✅ Completion Mode:");
+    const currentMode = localConfig.completion_mode ||
+        (qaPmTeams.length > 0 ? "upstream_strict" : "simple");
+    const defaultModeIndex = currentMode === "simple"
+        ? 0
+        : currentMode === "strict_review"
+            ? 1
+            : currentMode === "upstream_strict"
+                ? 2
+                : 3;
+    const modeResponse = await prompts({
+        type: "select",
+        name: "mode",
+        message: "How should tasks be completed?",
+        choices: [
+            {
+                title: "Simple",
+                value: "simple",
+                description: "Mark task as done directly",
+            },
+            {
+                title: "Strict Review",
+                value: "strict_review",
+                description: "Mark task to dev team's testing status",
+            },
+            {
+                title: "Upstream Strict (recommended with QA/PM)",
+                value: "upstream_strict",
+                description: "Done + parent to testing, fallback to testing if no parent",
+            },
+            {
+                title: "Upstream Not Strict",
+                value: "upstream_not_strict",
+                description: "Done + parent to testing, no fallback",
+            },
+        ],
+        initial: defaultModeIndex,
+    });
+    const completionMode = modeResponse.mode || (qaPmTeams.length > 0 ? "upstream_strict" : "simple");
+    // Update local config
+    localConfig.team = devTeamKey;
+    localConfig.dev_testing_status = devTestingStatus;
+    localConfig.qa_pm_teams = qaPmTeams.length > 0 ? qaPmTeams : undefined;
+    localConfig.completion_mode = completionMode;
+    // Remove deprecated fields
+    delete localConfig.teams;
+    delete localConfig.qa_pm_team;
     await saveLocalConfig(localConfig);
     console.log("\n✅ Teams updated:");
-    if (localConfig.teams) {
-        console.log(`  Selected: ${localConfig.teams.join(", ")}`);
-        console.log(`  Primary: ${localConfig.team}`);
+    console.log(`  Dev Team: ${devTeam?.name || devTeamKey}`);
+    if (devTestingStatus) {
+        console.log(`  Dev Testing Status: ${devTestingStatus}`);
     }
-    else {
-        console.log(`  Team: ${localConfig.team}`);
+    console.log(`  Completion Mode: ${completionMode}`);
+    if (qaPmTeams.length > 0) {
+        console.log("  QA/PM Teams:");
+        for (const qp of qaPmTeams) {
+            console.log(`    - ${qp.team}: ${qp.testing_status}`);
+        }
     }
 }

package/dist/scripts/done-job.js

@@ -3,6 +3,86 @@ import { buildCompletionComment, getLatestCommit } from "./lib/git.js";
 import { addComment, getStatusTransitions, getWorkflowStates, updateIssueStatus, } from "./lib/linear.js";
 import { syncSingleIssue } from "./lib/sync.js";
 import { getLinearClient, loadConfig, loadCycleData, loadLocalConfig, } from "./utils.js";
+// Helper function to update parent issue to a specific status
+async function updateParentStatus(parentIssueId, targetStatus, _qaPmTeams, config) {
+    try {
+        const client = getLinearClient();
+        const searchResult = await client.searchIssues(parentIssueId);
+        const parentIssue = searchResult.nodes.find((issue) => issue.identifier === parentIssueId);
+        if (!parentIssue) {
+            return { success: false };
+        }
+        const parentTeam = await parentIssue.team;
+        if (!parentTeam) {
+            return { success: false };
+        }
+        // Find the matching team configuration
+        const teamEntries = Object.entries(config.teams);
+        const matchingTeamEntry = teamEntries.find(([_, t]) => t.id === parentTeam.id);
+        if (!matchingTeamEntry) {
+            return { success: false };
+        }
+        const [parentTeamKey] = matchingTeamEntry;
+        // Get workflow states for the parent's team
+        const parentStates = await getWorkflowStates(config, parentTeamKey);
+        const targetState = parentStates.find((s) => s.name === targetStatus);
+        if (!targetState) {
+            return { success: false };
+        }
+        // Update the parent issue
+        await client.updateIssue(parentIssue.id, {
+            stateId: targetState.id,
+        });
+        return { success: true, status: targetStatus };
+    }
+    catch (error) {
+        console.error("Failed to update parent issue:", error);
+        return { success: false };
+    }
+}
+// Helper function to update parent issue to testing status (uses QA team config)
+async function updateParentToTesting(parentIssueId, qaPmTeams, config) {
+    try {
+        const client = getLinearClient();
+        const searchResult = await client.searchIssues(parentIssueId);
+        const parentIssue = searchResult.nodes.find((issue) => issue.identifier === parentIssueId);
+        if (!parentIssue) {
+            return { success: false };
+        }
+        const parentTeam = await parentIssue.team;
+        if (!parentTeam) {
+            return { success: false };
+        }
+        // Find the matching QA/PM team configuration
+        const teamEntries = Object.entries(config.teams);
+        const matchingTeamEntry = teamEntries.find(([_, t]) => t.id === parentTeam.id);
+        if (!matchingTeamEntry) {
+            return { success: false };
+        }
+        const [parentTeamKey] = matchingTeamEntry;
+        // Find the QA/PM team config for this parent's team
+        const qaPmConfig = qaPmTeams.find((qp) => qp.team === parentTeamKey);
+        if (!qaPmConfig) {
+            // Parent's team is not in the configured QA/PM teams
+            return { success: false };
+        }
+        // Get workflow states for the parent's team
+        const parentStates = await getWorkflowStates(config, parentTeamKey);
+        const testingState = parentStates.find((s) => s.name === qaPmConfig.testing_status);
+        if (!testingState) {
+            return { success: false };
+        }
+        // Update the parent issue
+        await client.updateIssue(parentIssue.id, {
+            stateId: testingState.id,
+        });
+        return { success: true, testingStatus: qaPmConfig.testing_status };
+    }
+    catch (error) {
+        console.error("Failed to update parent issue:", error);
+        return { success: false };
+    }
+}
 function parseArgs(args) {
     let issueId;
     let message;
@@ -110,10 +190,84 @@ Examples:
         process.env.LINEAR_API_KEY &&
         statusSource === "remote") {
         const transitions = getStatusTransitions(config);
-        // Update issue to Done
-        const success = await updateIssueStatus(task.linearId, transitions.done, config, localConfig.team);
-        if (success) {
-            console.log(`Linear: ${task.id} → ${transitions.done}`);
+        // Determine completion mode
+        const completionMode = localConfig.completion_mode ||
+            (localConfig.qa_pm_teams && localConfig.qa_pm_teams.length > 0
+                ? "upstream_strict"
+                : "simple");
+        // Get the testing status to use (from dev team)
+        const devTestingStatus = localConfig.dev_testing_status || transitions.testing;
+        // Execute based on completion mode
+        let parentUpdateSuccess = false;
+        let parentTestingStatus;
+        switch (completionMode) {
+            case "simple": {
+                // Mark task as done
+                const success = await updateIssueStatus(task.linearId, transitions.done, config, localConfig.team);
+                if (success) {
+                    console.log(`Linear: ${task.id} → ${transitions.done}`);
+                }
+                // Also mark parent as done if exists
+                if (task.parentIssueId) {
+                    const result = await updateParentStatus(task.parentIssueId, transitions.done, localConfig.qa_pm_teams, config);
+                    if (result.success) {
+                        console.log(`Linear: Parent ${task.parentIssueId} → ${transitions.done}`);
+                    }
+                }
+                break;
+            }
+            case "strict_review": {
+                // Mark task to dev team's testing status
+                if (devTestingStatus) {
+                    const success = await updateIssueStatus(task.linearId, devTestingStatus, config, localConfig.team);
+                    if (success) {
+                        console.log(`Linear: ${task.id} → ${devTestingStatus}`);
+                    }
+                    // Also mark parent to testing if exists
+                    if (task.parentIssueId && localConfig.qa_pm_teams?.length) {
+                        const result = await updateParentToTesting(task.parentIssueId, localConfig.qa_pm_teams, config);
+                        if (result.success) {
+                            console.log(`Linear: Parent ${task.parentIssueId} → ${result.testingStatus}`);
+                        }
+                    }
+                }
+                else {
+                    console.warn("No dev testing status configured, falling back to done");
+                    const success = await updateIssueStatus(task.linearId, transitions.done, config, localConfig.team);
+                    if (success) {
+                        console.log(`Linear: ${task.id} → ${transitions.done}`);
+                    }
+                }
+                break;
+            }
+            case "upstream_strict":
+            case "upstream_not_strict": {
+                // First, mark as done
+                const doneSuccess = await updateIssueStatus(task.linearId, transitions.done, config, localConfig.team);
+                if (doneSuccess) {
+                    console.log(`Linear: ${task.id} → ${transitions.done}`);
+                }
+                // Try to update parent to testing
+                if (task.parentIssueId && localConfig.qa_pm_teams?.length) {
+                    const result = await updateParentToTesting(task.parentIssueId, localConfig.qa_pm_teams, config);
+                    parentUpdateSuccess = result.success;
+                    parentTestingStatus = result.testingStatus;
+                    if (parentUpdateSuccess) {
+                        console.log(`Linear: Parent ${task.parentIssueId} → ${parentTestingStatus}`);
+                    }
+                }
+                // Fallback logic for upstream_strict
+                if (completionMode === "upstream_strict" &&
+                    !parentUpdateSuccess &&
+                    devTestingStatus) {
+                    // No parent or parent update failed, fallback to testing
+                    const fallbackSuccess = await updateIssueStatus(task.linearId, devTestingStatus, config, localConfig.team);
+                    if (fallbackSuccess) {
+                        console.log(`Linear: ${task.id} → ${devTestingStatus} (fallback, no valid parent)`);
+                    }
+                }
+                break;
+            }
         }
         // Add comment with commit info
         if (commit) {
@@ -123,43 +277,6 @@ Examples:
                 console.log(`Linear: 已新增 commit 留言`);
             }
         }
-        // Update parent to Testing if exists
-        if (task.parentIssueId && transitions.testing) {
-            try {
-                const client = getLinearClient();
-                const searchResult = await client.searchIssues(task.parentIssueId);
-                const parentIssue = searchResult.nodes.find((issue) => issue.identifier === task.parentIssueId);
-                if (parentIssue) {
-                    const parentTeam = await parentIssue.team;
-                    if (parentTeam) {
-                        // Determine which team key to use for parent's workflow states
-                        // If qa_pm_team is configured and matches parent's team, use it
-                        // Otherwise, try to find the team key from config
-                        let parentTeamKey = localConfig.team;
-                        const teamEntries = Object.entries(config.teams);
-                        const matchingTeam = teamEntries.find(([_, t]) => t.id === parentTeam.id);
-                        if (matchingTeam) {
-                            parentTeamKey = matchingTeam[0];
-                        }
-                        else if (localConfig.qa_pm_team) {
-                            // Fallback to qa_pm_team if configured
-                            parentTeamKey = localConfig.qa_pm_team;
-                        }
-                        const parentStates = await getWorkflowStates(config, parentTeamKey);
-                        const testingState = parentStates.find((s) => s.name === transitions.testing);
-                        if (testingState) {
-                            await client.updateIssue(parentIssue.id, {
-                                stateId: testingState.id,
-                            });
-                            console.log(`Linear: Parent ${task.parentIssueId} → ${transitions.testing}`);
-                        }
-                    }
-                }
-            }
-            catch (parentError) {
-                console.error("Failed to update parent issue:", parentError);
-            }
-        }
     }
     else if (statusSource === "local") {
         console.log(`Local: ${task.id} marked as completed`);
@@ -187,9 +304,6 @@ Examples:
     if (aiMessage) {
         console.log(`AI: ${aiMessage}`);
     }
-    if (task.parentIssueId && config.status_transitions?.testing) {
-        console.log(`Parent: ${task.parentIssueId} → ${config.status_transitions.testing}`);
-    }
     console.log(`\n🎉 任務完成！`);
 }
 doneJob().catch(console.error);

package/dist/scripts/init.js

@@ -6,7 +6,7 @@ const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 import { decode, encode } from "@toon-format/toon";
 import prompts from "prompts";
-import { buildConfig, buildLocalConfig, findTeamKey, findUserKey, getDefaultStatusTransitions, } from "./lib/config-builder.js";
+import { buildConfig, buildLocalConfig, buildTeamsConfig, findTeamKey, findUserKey, getDefaultStatusTransitions, } from "./lib/config-builder.js";
 import { fileExists, getLinearClient, getPaths, } from "./utils.js";
 function parseArgs(args) {
     const options = { interactive: true };
@@ -82,77 +82,157 @@ async function promptForApiKey(options) {
     }
     return apiKey;
 }
-async function selectTeams(teams, options) {
-    let selectedTeams = [teams[0]];
-    let primaryTeam = teams[0];
+async function selectDevTeam(teams, options) {
+    let devTeam = teams[0];
     if (options.team) {
         const found = teams.find((t) => t.name.toLowerCase() === options.team?.toLowerCase());
         if (found) {
-            selectedTeams = [found];
-            primaryTeam = found;
+            devTeam = found;
         }
     }
     else if (options.interactive && teams.length > 1) {
+        console.log("\n👨‍💻 Dev Team Configuration:");
         const response = await prompts({
-            type: "multiselect",
-            name: "teamIds",
-            message: "Select teams to sync (space to select, enter to confirm):",
+            type: "select",
+            name: "teamId",
+            message: "Select your dev team (for work-on/done commands):",
             choices: teams.map((t) => ({ title: t.name, value: t.id })),
-            min: 1,
         });
-        if (response.teamIds && response.teamIds.length > 0) {
-            selectedTeams = teams.filter((t) => response.teamIds.includes(t.id));
-            if (selectedTeams.length > 1) {
-                const primaryResponse = await prompts({
-                    type: "select",
-                    name: "primaryTeamId",
-                    message: "Select your primary team (for work-on/done commands):",
-                    choices: selectedTeams.map((t) => ({ title: t.name, value: t.id })),
-                });
-                primaryTeam =
-                    selectedTeams.find((t) => t.id === primaryResponse.primaryTeamId) ||
-                        selectedTeams[0];
-            }
-            else {
-                primaryTeam = selectedTeams[0];
-            }
+        if (response.teamId) {
+            devTeam = teams.find((t) => t.id === response.teamId) || teams[0];
         }
     }
-    return { selected: selectedTeams, primary: primaryTeam };
+    return devTeam;
+}
+async function selectDevTestingStatus(devStates, options) {
+    if (!options.interactive || devStates.length === 0) {
+        return getDefaultStatusTransitions(devStates).testing;
+    }
+    console.log("\n🔍 Dev Team Testing/Review Status:");
+    const stateChoices = devStates.map((s) => ({
+        title: `${s.name} (${s.type})`,
+        value: s.name,
+    }));
+    const response = await prompts({
+        type: "select",
+        name: "testingStatus",
+        message: "Select testing/review status for dev team (used when strict_review mode):",
+        choices: [
+            { title: "(Skip - no testing status)", value: undefined },
+            ...stateChoices,
+        ],
+        initial: 0,
+    });
+    return response.testingStatus;
 }
-async function selectQaPmTeam(teams, primaryTeam, options) {
+async function selectQaPmTeams(teams, devTeam, teamStatesMap, teamsConfig, options) {
     // Only ask if there are multiple teams and interactive mode
     if (!options.interactive || teams.length <= 1) {
-        return undefined;
+        return [];
     }
-    // Filter out primary team from choices
-    const otherTeams = teams.filter((t) => t.id !== primaryTeam.id);
+    // Filter out dev team from choices
+    const otherTeams = teams.filter((t) => t.id !== devTeam.id);
     if (otherTeams.length === 0) {
-        return undefined;
+        return [];
     }
-    console.log("\n🔗 QA/PM Team Configuration:");
+    console.log("\n🔗 QA/PM Teams Configuration:");
     const response = await prompts({
-        type: "select",
-        name: "qaPmTeamId",
-        message: "Select QA/PM team (for cross-team parent issue updates):",
+        type: "multiselect",
+        name: "qaPmTeamIds",
+        message: "Select QA/PM teams for cross-team parent updates (space to select, enter to confirm):",
         choices: [
-            {
-                title: "(None - skip)",
-                value: undefined,
-                description: "No cross-team parent updates",
-            },
             ...otherTeams.map((t) => ({
                 title: t.name,
                 value: t.id,
                 description: "Parent issues in this team will be updated to Testing",
             })),
         ],
-        initial: 0,
+        hint: "- Press space to select, enter to confirm. Leave empty to skip.",
     });
-    if (!response.qaPmTeamId) {
-        return undefined;
+    if (!response.qaPmTeamIds || response.qaPmTeamIds.length === 0) {
+        return [];
+    }
+    // For each selected QA/PM team, select its testing status
+    const qaPmTeams = [];
+    for (const teamId of response.qaPmTeamIds) {
+        const team = teams.find((t) => t.id === teamId);
+        if (!team)
+            continue;
+        const teamStates = teamStatesMap.get(teamId) || [];
+        const defaults = getDefaultStatusTransitions(teamStates);
+        // Find team key
+        const teamKey = Object.entries(teamsConfig).find(([_, t]) => t.id === teamId)?.[0] ||
+            team.name.toLowerCase().replace(/[^a-z0-9]/g, "_");
+        if (teamStates.length === 0) {
+            // No states available, use default
+            if (defaults.testing) {
+                qaPmTeams.push({
+                    team: teamKey,
+                    testing_status: defaults.testing,
+                });
+            }
+            continue;
+        }
+        const stateChoices = teamStates.map((s) => ({
+            title: `${s.name} (${s.type})`,
+            value: s.name,
+        }));
+        const statusResponse = await prompts({
+            type: "select",
+            name: "testingStatus",
+            message: `Select testing status for ${team.name}:`,
+            choices: [
+                { title: "(Skip this team)", value: undefined },
+                ...stateChoices,
+            ],
+            initial: defaults.testing
+                ? stateChoices.findIndex((c) => c.value === defaults.testing) + 1
+                : 0,
+        });
+        if (statusResponse.testingStatus) {
+            qaPmTeams.push({
+                team: teamKey,
+                testing_status: statusResponse.testingStatus,
+            });
+        }
     }
-    return teams.find((t) => t.id === response.qaPmTeamId);
+    return qaPmTeams;
+}
+async function selectCompletionMode(hasQaPmTeams, options) {
+    if (!options.interactive) {
+        return hasQaPmTeams ? "upstream_strict" : "simple";
+    }
+    console.log("\n✅ Completion Mode Configuration:");
+    const defaultMode = hasQaPmTeams ? 2 : 0; // upstream_strict if has QA/PM teams, else simple
+    const response = await prompts({
+        type: "select",
+        name: "mode",
+        message: "How should tasks be completed?",
+        choices: [
+            {
+                title: "Simple",
+                value: "simple",
+                description: "Mark task as done directly",
+            },
+            {
+                title: "Strict Review",
+                value: "strict_review",
+                description: "Mark task to dev team's testing status",
+            },
+            {
+                title: "Upstream Strict (recommended with QA/PM)",
+                value: "upstream_strict",
+                description: "Done + parent to testing, fallback to testing if no parent",
+            },
+            {
+                title: "Upstream Not Strict",
+                value: "upstream_not_strict",
+                description: "Done + parent to testing, no fallback",
+            },
+        ],
+        initial: defaultMode,
+    });
+    return response.mode || (hasQaPmTeams ? "upstream_strict" : "simple");
 }
 async function selectUser(users, options) {
     let currentUser = users[0];
@@ -220,19 +300,14 @@ async function selectStatusSource(options) {
     });
     return response.statusSource || "remote";
 }
-async function selectStatusMappings(devStates, qaStates, options) {
+async function selectStatusMappings(devStates, options) {
     // Use dev team states for todo, in_progress, done, blocked
-    // Use qa team states for testing (fallback to dev team if not set)
+    // Testing status is now configured separately (dev_testing_status and qa_pm_teams)
     const devDefaults = getDefaultStatusTransitions(devStates);
-    const testingStates = qaStates && qaStates.length > 0 ? qaStates : devStates;
-    const testingDefaults = getDefaultStatusTransitions(testingStates);
     if (!options.interactive || devStates.length === 0) {
-        return {
-            ...devDefaults,
-            testing: testingDefaults.testing,
-        };
+        return devDefaults;
     }
-    console.log("\n📊 Configure status mappings:");
+    console.log("\n📊 Configure status mappings (dev team):");
     const devStateChoices = devStates.map((s) => ({
         title: `${s.name} (${s.type})`,
         value: s.name,
@@ -258,29 +333,8 @@ async function selectStatusMappings(devStates, qaStates, options) {
         choices: devStateChoices,
         initial: devStateChoices.findIndex((c) => c.value === devDefaults.done),
     });
-    // Testing uses qa team states (or dev team if qa not set)
-    const testingStateChoices = testingStates.map((s) => ({
-        title: `${s.name} (${s.type})`,
-        value: s.name,
-    }));
-    const testingChoices = [
-        { title: "(None)", value: undefined },
-        ...testingStateChoices,
-    ];
-    const testingMessage = qaStates && qaStates.length > 0
-        ? 'Select status for "Testing" (from QA team, for parent tasks):'
-        : 'Select status for "Testing" (optional, for parent tasks):';
-    const testingResponse = await prompts({
-        type: "select",
-        name: "testing",
-        message: testingMessage,
-        choices: testingChoices,
-        initial: testingDefaults.testing
-            ? testingChoices.findIndex((c) => c.value === testingDefaults.testing)
-            : 0,
-    });
     const blockedChoices = [
-        { title: "(None)", value: undefined },
+        { title: "(Skip - no blocked status)", value: undefined },
         ...devStateChoices,
     ];
     const blockedResponse = await prompts({
@@ -296,7 +350,7 @@ async function selectStatusMappings(devStates, qaStates, options) {
         todo: todoResponse.todo || devDefaults.todo,
         in_progress: inProgressResponse.in_progress || devDefaults.in_progress,
         done: doneResponse.done || devDefaults.done,
-        testing: testingResponse.testing,
+        testing: devDefaults.testing, // Will be overridden by dev_testing_status in LocalConfig
         blocked: blockedResponse.blocked,
     };
 }
@@ -536,12 +590,9 @@ async function init() {
         console.error("Error: No teams found in your Linear workspace.");
         process.exit(1);
     }
-    // Select teams
-    const { selected: selectedTeams, primary: primaryTeam } = await selectTeams(teams, options);
-    console.log(`  Teams: ${selectedTeams.map((t) => t.name).join(", ")}`);
-    if (selectedTeams.length > 1) {
-        console.log(`  Primary: ${primaryTeam.name}`);
-    }
+    // Select dev team (single selection)
+    const devTeam = await selectDevTeam(teams, options);
+    console.log(`  Dev Team: ${devTeam.name}`);
     // Fetch data from ALL teams (not just primary) to support cross-team operations
     console.log(`  Fetching data from ${teams.length} teams...`);
     // Collect users from all teams, but labels only from primary team
@@ -563,8 +614,8 @@ async function init() {
                     allUsers.push(user);
                 }
             }
-            // Labels: only from primary team (dev team)
-            if (team.id === primaryTeam.id) {
+            // Labels: only from dev team
+            if (team.id === devTeam.id) {
                 const labelsData = await client.issueLabels({
                     filter: { team: { id: { eq: team.id } } },
                 });
@@ -589,7 +640,7 @@ async function init() {
             }
             teamStatesMap.set(team.id, teamStates);
         }
-        catch (error) {
+        catch {
             console.warn(`  ⚠ Could not fetch data for team ${team.name}, skipping...`);
         }
     }
@@ -597,34 +648,34 @@ async function init() {
     const labels = allLabels;
     const states = allStates;
     // Get team-specific states for status mapping
-    const primaryTeamStates = teamStatesMap.get(primaryTeam.id) || [];
+    const devTeamStates = teamStatesMap.get(devTeam.id) || [];
     console.log(`  Users: ${users.length}`);
-    console.log(`  Labels: ${labels.length} (from ${primaryTeam.name})`);
+    console.log(`  Labels: ${labels.length} (from ${devTeam.name})`);
     console.log(`  Workflow states: ${states.length}`);
-    // Get cycle from primary team (for current work tracking)
-    const selectedTeam = await client.team(primaryTeam.id);
+    // Get cycle from dev team (for current work tracking)
+    const selectedTeam = await client.team(devTeam.id);
     const currentCycle = (await selectedTeam.activeCycle);
     // User selections
     const currentUser = await selectUser(users, options);
     const defaultLabel = await selectLabelFilter(labels, options);
     const statusSource = await selectStatusSource(options);
-    const qaPmTeam = await selectQaPmTeam(teams, primaryTeam, options);
-    // Get qa team states for testing status mapping (fallback to primary team if not set)
-    const qaTeamStates = qaPmTeam ? teamStatesMap.get(qaPmTeam.id) : undefined;
-    const statusTransitions = await selectStatusMappings(primaryTeamStates, qaTeamStates, options);
+    // Status transitions for dev team (todo, in_progress, done, blocked)
+    const statusTransitions = await selectStatusMappings(devTeamStates, options);
+    // Dev team testing status (for strict_review mode)
+    const devTestingStatus = await selectDevTestingStatus(devTeamStates, options);
+    // Build preliminary teams config for selectQaPmTeams
+    const teamsConfig = buildTeamsConfig(teams);
+    // QA/PM teams selection (multiple, each with its own testing status)
+    const qaPmTeams = await selectQaPmTeams(teams, devTeam, teamStatesMap, teamsConfig, options);
+    // Completion mode selection
+    const completionMode = await selectCompletionMode(qaPmTeams.length > 0, options);
     // Build config
     const config = buildConfig(teams, users, labels, states, statusTransitions, currentCycle ?? undefined);
     // Find keys
     const currentUserKey = findUserKey(config.users, currentUser.id);
-    const primaryTeamKey = findTeamKey(config.teams, primaryTeam.id);
-    const selectedTeamKeys = selectedTeams
-        .map((team) => findTeamKey(config.teams, team.id))
-        .filter((key) => key !== undefined);
-    const qaPmTeamKey = qaPmTeam
-        ? findTeamKey(config.teams, qaPmTeam.id)
-        : undefined;
-    const localConfig = buildLocalConfig(currentUserKey, primaryTeamKey, selectedTeamKeys, defaultLabel, undefined, // excludeLabels
-    statusSource, qaPmTeamKey);
+    const devTeamKey = findTeamKey(config.teams, devTeam.id);
+    const localConfig = buildLocalConfig(currentUserKey, devTeamKey, devTestingStatus, qaPmTeams, completionMode, defaultLabel, undefined, // excludeLabels
+    statusSource);
     // Write config files
     console.log("\n📝 Writing configuration files...");
     await fs.mkdir(paths.baseDir, { recursive: true });
@@ -659,10 +710,12 @@ async function init() {
                     localConfig.current_user = existingLocal.current_user;
                 if (existingLocal.team)
                     localConfig.team = existingLocal.team;
-                if (existingLocal.teams)
-                    localConfig.teams = existingLocal.teams;
-                if (existingLocal.qa_pm_team)
-                    localConfig.qa_pm_team = existingLocal.qa_pm_team;
+                if (existingLocal.dev_testing_status)
+                    localConfig.dev_testing_status = existingLocal.dev_testing_status;
+                if (existingLocal.qa_pm_teams)
+                    localConfig.qa_pm_teams = existingLocal.qa_pm_teams;
+                if (existingLocal.completion_mode)
+                    localConfig.completion_mode = existingLocal.completion_mode;
                 if (existingLocal.label)
                     localConfig.label = existingLocal.label;
                 if (existingLocal.exclude_labels)
@@ -684,15 +737,19 @@ async function init() {
     // Summary
     console.log("\n✅ Initialization complete!\n");
     console.log("Configuration summary:");
-    console.log(`  Teams: ${selectedTeams.map((t) => t.name).join(", ")}`);
-    if (selectedTeams.length > 1) {
-        console.log(`  Primary team: ${primaryTeam.name}`);
-    }
+    console.log(`  Dev Team: ${devTeam.name}`);
     console.log(`  User: ${currentUser.displayName || currentUser.name} (${currentUser.email})`);
     console.log(`  Label filter: ${defaultLabel || "(none)"}`);
     console.log(`  Status source: ${statusSource === "local" ? "local (use 'sync --update' to push)" : "remote (immediate sync)"}`);
-    if (qaPmTeam) {
-        console.log(`  QA/PM team: ${qaPmTeam.name}`);
+    console.log(`  Completion mode: ${completionMode}`);
+    if (devTestingStatus) {
+        console.log(`  Dev testing status: ${devTestingStatus}`);
+    }
+    if (qaPmTeams.length > 0) {
+        console.log(`  QA/PM teams:`);
+        for (const qaPmTeam of qaPmTeams) {
+            console.log(`    - ${qaPmTeam.team}: ${qaPmTeam.testing_status}`);
+        }
     }
     console.log(`  (Use 'ttt config filters' to set excluded labels/users)`);
     if (currentCycle) {
@@ -702,9 +759,6 @@ async function init() {
     console.log(`    Todo: ${statusTransitions.todo}`);
     console.log(`    In Progress: ${statusTransitions.in_progress}`);
     console.log(`    Done: ${statusTransitions.done}`);
-    if (statusTransitions.testing) {
-        console.log(`    Testing: ${statusTransitions.testing}`);
-    }
     if (statusTransitions.blocked) {
         console.log(`    Blocked: ${statusTransitions.blocked}`);
     }

package/dist/scripts/lib/config-builder.d.ts

@@ -1,4 +1,4 @@
-import type { Config, LocalConfig, StatusTransitions, TeamConfig, UserConfig, LabelConfig } from "../utils.js";
+import type { CompletionMode, Config, LocalConfig, QaPmTeamConfig, StatusTransitions, TeamConfig, UserConfig, LabelConfig } from "../utils.js";
 export interface LinearTeam {
     id: string;
     name: string;
@@ -38,4 +38,4 @@ export declare function getDefaultStatusTransitions(states: LinearState[]): Stat
 export declare function buildConfig(teams: LinearTeam[], users: LinearUser[], labels: LinearLabel[], states: LinearState[], statusTransitions: StatusTransitions, currentCycle?: LinearCycle): Config;
 export declare function findUserKey(usersConfig: Record<string, UserConfig>, userId: string): string;
 export declare function findTeamKey(teamsConfig: Record<string, TeamConfig>, teamId: string): string;
-export declare function buildLocalConfig(currentUserKey: string, primaryTeamKey: string, selectedTeamKeys: string[], defaultLabel?: string, excludeLabels?: string[], statusSource?: "remote" | "local", qaPmTeam?: string): LocalConfig;
+export declare function buildLocalConfig(currentUserKey: string, devTeamKey: string, devTestingStatus?: string, qaPmTeams?: QaPmTeamConfig[], completionMode?: CompletionMode, defaultLabel?: string, excludeLabels?: string[], statusSource?: "remote" | "local"): LocalConfig;

package/dist/scripts/lib/config-builder.js

@@ -108,12 +108,13 @@ export function findTeamKey(teamsConfig, teamId) {
     return (Object.entries(teamsConfig).find(([_, t]) => t.id === teamId)?.[0] ||
         Object.keys(teamsConfig)[0]);
 }
-export function buildLocalConfig(currentUserKey, primaryTeamKey, selectedTeamKeys, defaultLabel, excludeLabels, statusSource, qaPmTeam) {
+export function buildLocalConfig(currentUserKey, devTeamKey, devTestingStatus, qaPmTeams, completionMode, defaultLabel, excludeLabels, statusSource) {
     return {
         current_user: currentUserKey,
-        team: primaryTeamKey,
-        teams: selectedTeamKeys.length > 1 ? selectedTeamKeys : undefined,
-        qa_pm_team: qaPmTeam,
+        team: devTeamKey,
+        dev_testing_status: devTestingStatus,
+        qa_pm_teams: qaPmTeams && qaPmTeams.length > 0 ? qaPmTeams : undefined,
+        completion_mode: completionMode,
         label: defaultLabel,
         exclude_labels: excludeLabels && excludeLabels.length > 0 ? excludeLabels : undefined,
         status_source: statusSource,

package/dist/scripts/lib/images.js

@@ -24,12 +24,13 @@ export function extractLinearImageUrls(text) {
         /(https?:\/\/uploads\.linear\.app\/[^\s)>\]]+)/g, // Plain Linear upload URLs
     ];
     for (const pattern of patterns) {
-        let match;
-        while ((match = pattern.exec(text)) !== null) {
+        let match = pattern.exec(text);
+        while (match) {
             const url = match[1];
             if (isLinearImageUrl(url) && !urls.includes(url)) {
                 urls.push(url);
             }
+            match = pattern.exec(text);
         }
     }
     return urls;
@@ -66,7 +67,7 @@ export async function downloadLinearFile(url, issueId, attachmentId, outputDir)
         // Linear files require authentication
         const headers = {};
         if (process.env.LINEAR_API_KEY) {
-            headers["Authorization"] = process.env.LINEAR_API_KEY;
+            headers.Authorization = process.env.LINEAR_API_KEY;
         }
         const response = await fetch(url, { headers });
         if (!response.ok) {

package/dist/scripts/utils.d.ts

@@ -35,6 +35,11 @@ export interface StatusTransitions {
     testing?: string;
     blocked?: string;
 }
+export interface QaPmTeamConfig {
+    team: string;
+    testing_status: string;
+}
+export type CompletionMode = "simple" | "strict_review" | "upstream_strict" | "upstream_not_strict";
 export interface Config {
     teams: Record<string, TeamConfig>;
     users: Record<string, UserConfig>;
@@ -94,11 +99,14 @@ export interface CycleData {
 export interface LocalConfig {
     current_user: string;
     team: string;
-    teams?: string[];
-    qa_pm_team?: string;
+    dev_testing_status?: string;
+    qa_pm_teams?: QaPmTeamConfig[];
+    completion_mode?: CompletionMode;
     exclude_labels?: string[];
     label?: string;
     status_source?: "remote" | "local";
+    teams?: string[];
+    qa_pm_team?: string;
 }
 export declare function fileExists(filePath: string): Promise<boolean>;
 export declare function loadConfig(): Promise<Config>;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "team-toon-tack",
-	"version": "2.0.3",
+	"version": "2.1.0",
 	"description": "Linear task sync & management CLI with TOON format",
 	"type": "module",
 	"bin": {