chore(thread): remove --from-file mutation option

Author: gnapse

README.md

@@ -58,19 +58,17 @@ Run `tw --help` or `tw <command> --help` for more options.
 
 ## Inbox State Workflows
 
-Thread inbox-state commands support single refs, bulk refs, and file input:
+Thread inbox-state commands support single refs and bulk refs:
 
 ```bash
 tw thread reopen 123 456 --yes                    # bulk reopen
-tw thread unread --from-file ids.txt --yes        # refs from file
 tw thread restore 123 --unread --json             # reopen + unread with JSON output
 tw thread done 123 --dry-run                      # preview archive operation
 ```
 
 Each mutation command supports:
 
 ```bash
---from-file <path>   # one thread ref per line
 --yes                # skip bulk confirmation
 --dry-run            # preview without changing state
 --json               # include before/after state per thread

src/__tests__/thread.test.ts

@@ -1,6 +1,3 @@
-import { mkdtemp, writeFile } from 'node:fs/promises'
-import { tmpdir } from 'node:os'
-import { join } from 'node:path'
 import { Command } from 'commander'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 
@@ -291,62 +288,6 @@ describe('thread reversible inbox mutations', () => {
         logSpy.mockRestore()
     })
 
-    it('supports loading refs from --from-file', async () => {
-        const fake = createFakeThreadClient({
-            threads: [
-                {
-                    id: 100,
-                    title: 'Thread 100',
-                    channelId: 10,
-                    workspaceId: 1,
-                    isArchived: false,
-                    inInbox: true,
-                    closed: null,
-                    lastUpdated: new Date('2026-03-03T17:00:00.000Z'),
-                    lastObjIndex: 4,
-                },
-                {
-                    id: 101,
-                    title: 'Thread 101',
-                    channelId: 10,
-                    workspaceId: 1,
-                    isArchived: false,
-                    inInbox: true,
-                    closed: null,
-                    lastUpdated: new Date('2026-03-03T17:00:00.000Z'),
-                    lastObjIndex: 6,
-                },
-            ],
-            unreadByWorkspace: { 1: [101] },
-        })
-        apiMocks.getTwistClient.mockResolvedValue(fake.client)
-
-        const dir = await mkdtemp(join(tmpdir(), 'tw-thread-'))
-        const refsFile = join(dir, 'refs.txt')
-        await writeFile(refsFile, '100\n101\n')
-
-        const program = createProgram()
-        const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
-
-        await program.parseAsync([
-            'node',
-            'tw',
-            'thread',
-            'unread',
-            '--from-file',
-            refsFile,
-            '--yes',
-            '--json',
-        ])
-
-        const output = parseLastJsonLog(logSpy)
-        expect(output.results).toHaveLength(2)
-        expect(output.errors).toEqual([])
-        expect(output.results.every((r) => r.after.isUnread === true)).toBe(true)
-
-        logSpy.mockRestore()
-    })
-
     it('supports dry-run without mutating thread state', async () => {
         const fake = createFakeThreadClient({
             threads: [

src/commands/thread/mutate.ts

@@ -1,4 +1,3 @@
-import { readFile } from 'node:fs/promises'
 import { createInterface } from 'node:readline/promises'
 import type { TwistApi } from '@doist/twist-sdk'
 import { Command } from 'commander'
@@ -10,7 +9,6 @@ import { resolveThreadId } from '../../lib/refs.js'
 import { pluralize } from './helpers.js'
 
 type ThreadMutationOptions = MutationOptions & {
-    fromFile?: string
     yes?: boolean
     json?: boolean
 }
@@ -155,25 +153,6 @@ async function getUnreadThreadIds(client: TwistApi, workspaceId: number): Promis
     return new Set(unread.map((item) => item.threadId))
 }
 
-async function loadThreadRefs(
-    fromArgs: string[] | undefined,
-    fromFilePath?: string,
-): Promise<string[]> {
-    const refs = [...(fromArgs ?? [])]
-
-    if (!fromFilePath) {
-        return refs
-    }
-
-    const fileContent = await readFile(fromFilePath, 'utf8')
-    const fromFile = fileContent
-        .split(/\r?\n/)
-        .map((line) => line.trim())
-        .filter((line) => line !== '' && !line.startsWith('#'))
-
-    return [...refs, ...fromFile]
-}
-
 async function confirmBulkMutation(count: number, mutationLabel: string): Promise<boolean> {
     if (count <= 1) {
         return true
@@ -282,10 +261,10 @@ async function executeThreadMutation(
     mutation: ThreadMutation,
     mutationLabel: string,
 ): Promise<void> {
-    const inputRefs = await loadThreadRefs(refs, options.fromFile)
+    const inputRefs = [...(refs ?? [])]
 
     if (inputRefs.length === 0) {
-        throw new Error('No thread references provided. Pass refs and/or --from-file.')
+        throw new Error('No thread references provided.')
     }
 
     if (inputRefs.length > 1 && !options.dryRun && !options.yes) {
@@ -361,7 +340,6 @@ async function executeThreadMutation(
 
 export function withThreadMutationOptions(command: Command): Command {
     return command
-        .option('--from-file <path>', 'Read thread refs from file (one ref per line)')
         .option('--yes', 'Skip confirmation for bulk operations')
         .option('--dry-run', 'Show what would happen without executing')
         .option('--json', 'Output mutation results as JSON')

src/lib/skills/content.ts

@@ -64,11 +64,10 @@ tw thread read <ref>             # Mark thread as read
 tw thread unread <ref>           # Mark thread as unread
 tw thread restore <ref> --unread # Reopen and mark unread
 tw thread reopen 123 456 --yes   # Bulk operation with multiple refs
-tw thread unread --from-file ids.txt --yes  # Load refs from file
 \`\`\`
 
 Default \`--notify\` is EVERYONE_IN_THREAD. Options: EVERYONE, EVERYONE_IN_THREAD, or comma-separated user ID refs.
-Thread mutation commands (\`done\`, \`reopen\`, \`read\`, \`unread\`, \`restore\`) support \`--from-file\`, \`--yes\`, \`--dry-run\`, and \`--json\`.
+Thread mutation commands (\`done\`, \`reopen\`, \`read\`, \`unread\`, \`restore\`) support \`--yes\`, \`--dry-run\`, and \`--json\`.
 
 ## Conversations (DMs/Groups)