docrev 0.9.11 → 0.9.14

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (138) hide show
  1. package/.claude/settings.local.json +9 -9
  2. package/.gitattributes +1 -1
  3. package/CHANGELOG.md +149 -149
  4. package/PLAN-tables-and-postprocess.md +850 -850
  5. package/README.md +391 -391
  6. package/bin/rev.js +11 -11
  7. package/bin/rev.ts +145 -145
  8. package/completions/rev.bash +127 -127
  9. package/completions/rev.ps1 +210 -210
  10. package/completions/rev.zsh +207 -207
  11. package/dev_notes/stress2/build_adversarial.ts +186 -186
  12. package/dev_notes/stress2/drift_matcher.ts +62 -62
  13. package/dev_notes/stress2/probe_anchors.ts +35 -35
  14. package/dev_notes/stress2/project/discussion.before.md +3 -3
  15. package/dev_notes/stress2/project/discussion.md +3 -3
  16. package/dev_notes/stress2/project/methods.before.md +20 -20
  17. package/dev_notes/stress2/project/methods.md +20 -20
  18. package/dev_notes/stress2/project/rev.yaml +5 -5
  19. package/dev_notes/stress2/project/sections.yaml +4 -4
  20. package/dev_notes/stress2/sections.yaml +5 -5
  21. package/dev_notes/stress2/trace_placement.ts +50 -50
  22. package/dev_notes/stresstest_boundaries.ts +27 -27
  23. package/dev_notes/stresstest_drift_apply.ts +43 -43
  24. package/dev_notes/stresstest_drift_compare.ts +43 -43
  25. package/dev_notes/stresstest_drift_v2.ts +54 -54
  26. package/dev_notes/stresstest_inspect.ts +54 -54
  27. package/dev_notes/stresstest_pstyle.ts +55 -55
  28. package/dev_notes/stresstest_section_debug.ts +23 -23
  29. package/dev_notes/stresstest_split.ts +70 -70
  30. package/dev_notes/stresstest_trace.ts +19 -19
  31. package/dev_notes/stresstest_verify_no_overwrite.ts +40 -40
  32. package/dist/lib/build.d.ts +50 -1
  33. package/dist/lib/build.d.ts.map +1 -1
  34. package/dist/lib/build.js +80 -30
  35. package/dist/lib/build.js.map +1 -1
  36. package/dist/lib/commands/build.d.ts.map +1 -1
  37. package/dist/lib/commands/build.js +38 -5
  38. package/dist/lib/commands/build.js.map +1 -1
  39. package/dist/lib/commands/utilities.js +164 -164
  40. package/dist/lib/commands/word-tools.js +8 -8
  41. package/dist/lib/grammar.js +3 -3
  42. package/dist/lib/import.d.ts.map +1 -1
  43. package/dist/lib/import.js +146 -24
  44. package/dist/lib/import.js.map +1 -1
  45. package/dist/lib/pdf-comments.js +44 -44
  46. package/dist/lib/plugins.js +57 -57
  47. package/dist/lib/pptx-themes.js +115 -115
  48. package/dist/lib/spelling.js +2 -2
  49. package/dist/lib/templates.js +387 -387
  50. package/dist/lib/themes.js +51 -51
  51. package/dist/lib/types.d.ts +20 -0
  52. package/dist/lib/types.d.ts.map +1 -1
  53. package/dist/lib/word-extraction.d.ts +6 -0
  54. package/dist/lib/word-extraction.d.ts.map +1 -1
  55. package/dist/lib/word-extraction.js +46 -3
  56. package/dist/lib/word-extraction.js.map +1 -1
  57. package/dist/lib/wordcomments.d.ts.map +1 -1
  58. package/dist/lib/wordcomments.js +23 -5
  59. package/dist/lib/wordcomments.js.map +1 -1
  60. package/eslint.config.js +27 -27
  61. package/lib/anchor-match.ts +276 -276
  62. package/lib/annotations.ts +644 -644
  63. package/lib/build.ts +1300 -1227
  64. package/lib/citations.ts +160 -160
  65. package/lib/commands/build.ts +833 -801
  66. package/lib/commands/citations.ts +515 -515
  67. package/lib/commands/comments.ts +1050 -1050
  68. package/lib/commands/context.ts +174 -174
  69. package/lib/commands/core.ts +309 -309
  70. package/lib/commands/doi.ts +435 -435
  71. package/lib/commands/file-ops.ts +372 -372
  72. package/lib/commands/history.ts +320 -320
  73. package/lib/commands/index.ts +87 -87
  74. package/lib/commands/init.ts +259 -259
  75. package/lib/commands/merge-resolve.ts +378 -378
  76. package/lib/commands/preview.ts +178 -178
  77. package/lib/commands/project-info.ts +244 -244
  78. package/lib/commands/quality.ts +517 -517
  79. package/lib/commands/response.ts +454 -454
  80. package/lib/commands/section-boundaries.ts +82 -82
  81. package/lib/commands/sections.ts +451 -451
  82. package/lib/commands/sync.ts +706 -706
  83. package/lib/commands/text-ops.ts +449 -449
  84. package/lib/commands/utilities.ts +448 -448
  85. package/lib/commands/verify-anchors.ts +272 -272
  86. package/lib/commands/word-tools.ts +340 -340
  87. package/lib/comment-realign.ts +517 -517
  88. package/lib/config.ts +84 -84
  89. package/lib/crossref.ts +781 -781
  90. package/lib/csl.ts +191 -191
  91. package/lib/dependencies.ts +98 -98
  92. package/lib/diff-engine.ts +465 -465
  93. package/lib/doi-cache.ts +115 -115
  94. package/lib/doi.ts +897 -897
  95. package/lib/equations.ts +506 -506
  96. package/lib/errors.ts +346 -346
  97. package/lib/format.ts +541 -541
  98. package/lib/git.ts +326 -326
  99. package/lib/grammar.ts +303 -303
  100. package/lib/image-registry.ts +180 -180
  101. package/lib/import.ts +911 -792
  102. package/lib/journals.ts +543 -543
  103. package/lib/merge.ts +633 -633
  104. package/lib/orcid.ts +144 -144
  105. package/lib/pdf-comments.ts +263 -263
  106. package/lib/pdf-import.ts +524 -524
  107. package/lib/plugins.ts +362 -362
  108. package/lib/postprocess.ts +188 -188
  109. package/lib/pptx-color-filter.lua +37 -37
  110. package/lib/pptx-template.ts +469 -469
  111. package/lib/pptx-themes.ts +483 -483
  112. package/lib/protect-restore.ts +520 -520
  113. package/lib/rate-limiter.ts +94 -94
  114. package/lib/response.ts +197 -197
  115. package/lib/restore-references.ts +240 -240
  116. package/lib/review.ts +327 -327
  117. package/lib/schema.ts +417 -417
  118. package/lib/scientific-words.ts +73 -73
  119. package/lib/sections.ts +335 -335
  120. package/lib/slides.ts +756 -756
  121. package/lib/spelling.ts +334 -334
  122. package/lib/templates.ts +526 -526
  123. package/lib/themes.ts +742 -742
  124. package/lib/trackchanges.ts +247 -247
  125. package/lib/tui.ts +450 -450
  126. package/lib/types.ts +550 -530
  127. package/lib/undo.ts +250 -250
  128. package/lib/utils.ts +69 -69
  129. package/lib/variables.ts +179 -179
  130. package/lib/word-extraction.ts +806 -759
  131. package/lib/word.ts +643 -643
  132. package/lib/wordcomments.ts +817 -798
  133. package/package.json +137 -137
  134. package/scripts/postbuild.js +28 -28
  135. package/skill/REFERENCE.md +431 -431
  136. package/skill/SKILL.md +258 -258
  137. package/tsconfig.json +26 -26
  138. package/types/index.d.ts +525 -525
@@ -1,378 +1,378 @@
1
- /**
2
- * MERGE, CONFLICTS, and MERGE-RESOLVE commands
3
- *
4
- * Commands for three-way merge of reviewer feedback and conflict resolution.
5
- * Split from sections.ts for maintainability.
6
- */
7
-
8
- import {
9
- chalk,
10
- fs,
11
- path,
12
- fmt,
13
- } from './context.js';
14
- import type { Command } from 'commander';
15
- import * as readline from 'readline';
16
-
17
- interface MergeOptions {
18
- base?: string;
19
- output?: string;
20
- names?: string;
21
- strategy: string;
22
- diffLevel: 'sentence' | 'word';
23
- dryRun?: boolean;
24
- sections?: boolean;
25
- }
26
-
27
- /**
28
- * Register merge, conflicts, and merge-resolve commands with the program
29
- */
30
- export function register(program: Command): void {
31
- // ==========================================================================
32
- // MERGE command - Combine feedback from multiple reviewers (three-way merge)
33
- // ==========================================================================
34
-
35
- program
36
- .command('merge')
37
- .description('Merge feedback from multiple Word documents using three-way merge')
38
- .argument('<docx...>', 'Word documents from reviewers')
39
- .option('-b, --base <file>', 'Base document (original sent to reviewers). Auto-detected if not specified.')
40
- .option('-o, --output <file>', 'Output file (default: writes to section files)')
41
- .option('--names <names>', 'Reviewer names (comma-separated, in order of docx files)')
42
- .option('--strategy <strategy>', 'Conflict resolution: first, latest, or interactive (default)', 'interactive')
43
- .option('--diff-level <level>', 'Diff granularity: sentence or word (default: sentence)', 'sentence')
44
- .option('--dry-run', 'Show conflicts without writing')
45
- .option('--sections', 'Split merged output back to section files')
46
- .action(async (docxFiles: string[], options: MergeOptions) => {
47
- const {
48
- mergeThreeWay,
49
- formatConflict,
50
- resolveConflict,
51
- getBaseDocument,
52
- checkBaseMatch,
53
- saveConflicts,
54
- } = await import('../merge.js');
55
-
56
- // Validate reviewer files exist
57
- for (const docx of docxFiles) {
58
- if (!fs.existsSync(docx)) {
59
- console.error(fmt.status('error', `Reviewer file not found: ${docx}`));
60
- process.exit(1);
61
- }
62
- }
63
-
64
- // Determine base document
65
- let basePath = options.base;
66
- let baseSource = 'specified';
67
-
68
- if (!basePath) {
69
- // Try to use .rev/base.docx
70
- const projectDir = process.cwd();
71
- basePath = getBaseDocument(projectDir) ?? undefined;
72
-
73
- if (basePath) {
74
- baseSource = 'auto (.rev/base.docx)';
75
- } else {
76
- console.log(chalk.yellow('\n No base document found in .rev/base.docx'));
77
- console.log(chalk.dim(' Tip: Run "rev build docx" to automatically save the base document.\n'));
78
- console.error(fmt.status('error', 'Base document required. Use --base <file.docx>'));
79
- process.exit(1);
80
- }
81
- }
82
-
83
- if (!fs.existsSync(basePath)) {
84
- console.error(fmt.status('error', `Base document not found: ${basePath}`));
85
- process.exit(1);
86
- }
87
-
88
- // Check similarity between base and reviewer docs
89
- const { matches, similarity } = await checkBaseMatch(basePath, docxFiles[0]);
90
- if (!matches) {
91
- console.log(chalk.yellow(`\n Warning: Base document may not match reviewer file (${Math.round(similarity * 100)}% similar)`));
92
- console.log(chalk.dim(' If this is wrong, use --base to specify the correct original document.\n'));
93
- }
94
-
95
- // Parse reviewer names
96
- const names = options.names
97
- ? options.names.split(',').map(n => n.trim())
98
- : docxFiles.map((f, i) => {
99
- // Try to extract name from filename (e.g., paper_reviewer_A.docx)
100
- const basename = path.basename(f, '.docx');
101
- const match = basename.match(/_([A-Za-z]+)$/);
102
- return match ? match[1] : `Reviewer ${i + 1}`;
103
- });
104
-
105
- // Pad names if needed
106
- while (names.length < docxFiles.length) {
107
- names.push(`Reviewer ${names.length + 1}`);
108
- }
109
-
110
- const reviewerDocs = docxFiles.map((p, i) => ({
111
- path: p,
112
- name: names[i],
113
- }));
114
-
115
- console.log(fmt.header('Three-Way Merge'));
116
- console.log();
117
- console.log(chalk.dim(` Base: ${path.basename(basePath)} (${baseSource})`));
118
- console.log(chalk.dim(` Reviewers: ${names.join(', ')}`));
119
- console.log(chalk.dim(` Diff level: ${options.diffLevel}`));
120
- console.log();
121
-
122
- const spin = fmt.spinner('Analyzing changes...').start();
123
-
124
- try {
125
- const { merged, conflicts, stats, baseText } = await mergeThreeWay(basePath, reviewerDocs, {
126
- diffLevel: options.diffLevel,
127
- });
128
-
129
- spin.stop();
130
-
131
- // Display stats
132
- console.log(fmt.table(['Metric', 'Count'], [
133
- ['Total changes', stats.totalChanges.toString()],
134
- ['Non-conflicting', stats.nonConflicting.toString()],
135
- ['Conflicts', stats.conflicts.toString()],
136
- ['Comments', stats.comments.toString()],
137
- ]));
138
- console.log();
139
-
140
- let finalMerged = merged;
141
-
142
- // Handle conflicts
143
- if (conflicts.length > 0) {
144
- console.log(chalk.yellow(`Found ${conflicts.length} conflict(s):\n`));
145
-
146
- if (options.strategy === 'first') {
147
- // Auto-resolve: take first reviewer's change
148
- for (const conflict of conflicts) {
149
- console.log(chalk.dim(` Conflict ${conflict.id}: using ${conflict.changes[0].reviewer}'s change`));
150
- resolveConflict(conflict, 0);
151
- }
152
- } else if (options.strategy === 'latest') {
153
- // Auto-resolve: take last reviewer's change
154
- for (const conflict of conflicts) {
155
- const lastIdx = conflict.changes.length - 1;
156
- console.log(chalk.dim(` Conflict ${conflict.id}: using ${conflict.changes[lastIdx].reviewer}'s change`));
157
- resolveConflict(conflict, lastIdx);
158
- }
159
- } else if (!options.dryRun) {
160
- // Interactive resolution
161
- for (let i = 0; i < conflicts.length; i++) {
162
- const conflict = conflicts[i];
163
- console.log(chalk.bold(`\nConflict ${i + 1}/${conflicts.length} (${conflict.id}):`));
164
- console.log(formatConflict(conflict, baseText));
165
- console.log();
166
-
167
- const rl = readline.createInterface({
168
- input: process.stdin,
169
- output: process.stdout,
170
- });
171
-
172
- const answer = await new Promise<string>((resolve) =>
173
- rl.question(chalk.cyan(` Choose (1-${conflict.changes.length}, s=skip): `), resolve)
174
- );
175
- rl.close();
176
-
177
- if (answer.toLowerCase() !== 's' && !isNaN(parseInt(answer))) {
178
- const choice = parseInt(answer) - 1;
179
- if (choice >= 0 && choice < conflict.changes.length) {
180
- resolveConflict(conflict, choice);
181
- console.log(chalk.green(` ✓ Applied: ${conflict.changes[choice].reviewer}'s change`));
182
- }
183
- } else {
184
- console.log(chalk.dim(' Skipped (will need manual resolution)'));
185
- }
186
- }
187
- }
188
-
189
- // Save unresolved conflicts for later
190
- const unresolved = conflicts.filter(c => c.resolved === null);
191
- if (unresolved.length > 0) {
192
- saveConflicts(process.cwd(), conflicts, basePath);
193
- console.log(chalk.yellow(`\n ${unresolved.length} unresolved conflict(s) saved to .rev/conflicts.json`));
194
- console.log(chalk.dim(' Run "rev conflicts" to view, "rev merge-resolve" to resolve'));
195
- }
196
- }
197
-
198
- // Write output
199
- if (!options.dryRun) {
200
- if (options.output) {
201
- // Write to single file
202
- fs.writeFileSync(options.output, finalMerged, 'utf-8');
203
- console.log(fmt.status('success', `Merged output written to ${options.output}`));
204
- } else if (options.sections) {
205
- // Split to section files (TODO: implement section splitting)
206
- console.log(chalk.yellow(' Section splitting not yet implemented'));
207
- console.log(chalk.dim(' Use -o to specify output file'));
208
- } else {
209
- // Default: write to merged.md
210
- const outPath = 'merged.md';
211
- fs.writeFileSync(outPath, finalMerged, 'utf-8');
212
- console.log(fmt.status('success', `Merged output written to ${outPath}`));
213
- }
214
-
215
- console.log();
216
- console.log(chalk.dim('Next steps:'));
217
- console.log(chalk.dim(' 1. rev review merged.md - Accept/reject changes'));
218
- console.log(chalk.dim(' 2. rev comments merged.md - Address comments'));
219
- if (conflicts.some(c => c.resolved === null)) {
220
- console.log(chalk.dim(' 3. rev merge-resolve - Resolve remaining conflicts'));
221
- }
222
- } else {
223
- console.log(fmt.status('info', 'Dry run - no output written'));
224
- }
225
- } catch (err) {
226
- spin.stop();
227
- const error = err as Error;
228
- console.error(fmt.status('error', error.message));
229
- if (process.env.DEBUG) console.error(error.stack);
230
- process.exit(1);
231
- }
232
- });
233
-
234
- // ==========================================================================
235
- // CONFLICTS command - List unresolved conflicts
236
- // ==========================================================================
237
-
238
- program
239
- .command('conflicts')
240
- .description('List unresolved merge conflicts')
241
- .action(async () => {
242
- const { loadConflicts, formatConflict } = await import('../merge.js');
243
- const projectDir = process.cwd();
244
- const data = loadConflicts(projectDir);
245
-
246
- if (!data) {
247
- console.log(fmt.status('info', 'No conflicts file found'));
248
- return;
249
- }
250
-
251
- const unresolved = data.conflicts.filter((c: any) => c.resolved === null);
252
-
253
- if (unresolved.length === 0) {
254
- console.log(fmt.status('success', 'All conflicts resolved!'));
255
- return;
256
- }
257
-
258
- console.log(fmt.header(`Unresolved Conflicts (${unresolved.length})`));
259
- console.log();
260
- console.log(chalk.dim(` Base: ${data.base}`));
261
- console.log(chalk.dim(` Merged: ${data.merged}`));
262
- console.log();
263
-
264
- for (const conflict of unresolved) {
265
- console.log(chalk.bold(`Conflict ${conflict.id}:`));
266
- // Show abbreviated info
267
- console.log(chalk.dim(` Original: "${conflict.original.slice(0, 50)}${conflict.original.length > 50 ? '...' : ''}"`));
268
- console.log(chalk.dim(` Options: ${conflict.changes.map((c: any) => c.reviewer).join(', ')}`));
269
- console.log();
270
- }
271
-
272
- console.log(chalk.dim('Run "rev merge-resolve" to resolve conflicts interactively'));
273
- });
274
-
275
- // ==========================================================================
276
- // MERGE-RESOLVE command - Interactively resolve merge conflicts
277
- // ==========================================================================
278
-
279
- program
280
- .command('merge-resolve')
281
- .alias('mresolve')
282
- .description('Resolve merge conflicts interactively')
283
- .option('--theirs', 'Accept all changes from last reviewer')
284
- .option('--ours', 'Accept all changes from first reviewer')
285
- .action(async (options: { theirs?: boolean; ours?: boolean }) => {
286
- const { loadConflicts, saveConflicts, clearConflicts, resolveConflict, formatConflict } = await import('../merge.js');
287
- const projectDir = process.cwd();
288
- const data = loadConflicts(projectDir);
289
-
290
- if (!data) {
291
- console.log(fmt.status('info', 'No conflicts to resolve'));
292
- return;
293
- }
294
-
295
- const unresolved = data.conflicts.filter((c: any) => c.resolved === null);
296
-
297
- if (unresolved.length === 0) {
298
- console.log(fmt.status('success', 'All conflicts already resolved!'));
299
- clearConflicts(projectDir);
300
- return;
301
- }
302
-
303
- console.log(fmt.header(`Resolving ${unresolved.length} Conflict(s)`));
304
- console.log();
305
-
306
- if (options.theirs) {
307
- // Accept all from last reviewer
308
- for (const conflict of unresolved) {
309
- const lastIdx = conflict.changes.length - 1;
310
- resolveConflict(conflict, lastIdx);
311
- console.log(chalk.dim(` ${conflict.id}: accepted ${conflict.changes[lastIdx].reviewer}'s change`));
312
- }
313
- saveConflicts(projectDir, data.conflicts, data.base);
314
- console.log(fmt.status('success', `Resolved ${unresolved.length} conflicts (--theirs)`));
315
- } else if (options.ours) {
316
- // Accept all from first reviewer
317
- for (const conflict of unresolved) {
318
- resolveConflict(conflict, 0);
319
- console.log(chalk.dim(` ${conflict.id}: accepted ${conflict.changes[0].reviewer}'s change`));
320
- }
321
- saveConflicts(projectDir, data.conflicts, data.base);
322
- console.log(fmt.status('success', `Resolved ${unresolved.length} conflicts (--ours)`));
323
- } else {
324
- // Interactive resolution
325
- // Read base text for context display
326
- let baseText = '';
327
- try {
328
- const { extractFromWord } = await import('../import.js');
329
- const { text } = await extractFromWord(data.base);
330
- baseText = text;
331
- } catch {
332
- // Can't read base, show without context
333
- }
334
-
335
- for (let i = 0; i < unresolved.length; i++) {
336
- const conflict = unresolved[i];
337
- console.log(chalk.bold(`\nConflict ${i + 1}/${unresolved.length} (${conflict.id}):`));
338
- console.log(formatConflict(conflict, baseText));
339
- console.log();
340
-
341
- const rl = readline.createInterface({
342
- input: process.stdin,
343
- output: process.stdout,
344
- });
345
-
346
- const answer = await new Promise<string>((resolve) =>
347
- rl.question(chalk.cyan(` Choose (1-${conflict.changes.length}, s=skip, q=quit): `), resolve)
348
- );
349
- rl.close();
350
-
351
- if (answer.toLowerCase() === 'q') {
352
- console.log(chalk.dim('\n Saving progress...'));
353
- break;
354
- }
355
-
356
- if (answer.toLowerCase() !== 's' && !isNaN(parseInt(answer))) {
357
- const choice = parseInt(answer) - 1;
358
- if (choice >= 0 && choice < conflict.changes.length) {
359
- resolveConflict(conflict, choice);
360
- console.log(chalk.green(` ✓ Applied: ${conflict.changes[choice].reviewer}'s change`));
361
- }
362
- } else {
363
- console.log(chalk.dim(' Skipped'));
364
- }
365
- }
366
-
367
- saveConflicts(projectDir, data.conflicts, data.base);
368
-
369
- const remaining = data.conflicts.filter((c: any) => c.resolved === null).length;
370
- if (remaining === 0) {
371
- console.log(fmt.status('success', '\nAll conflicts resolved!'));
372
- clearConflicts(projectDir);
373
- } else {
374
- console.log(chalk.yellow(`\n ${remaining} conflict(s) remaining`));
375
- }
376
- }
377
- });
378
- }
1
+ /**
2
+ * MERGE, CONFLICTS, and MERGE-RESOLVE commands
3
+ *
4
+ * Commands for three-way merge of reviewer feedback and conflict resolution.
5
+ * Split from sections.ts for maintainability.
6
+ */
7
+
8
+ import {
9
+ chalk,
10
+ fs,
11
+ path,
12
+ fmt,
13
+ } from './context.js';
14
+ import type { Command } from 'commander';
15
+ import * as readline from 'readline';
16
+
17
+ interface MergeOptions {
18
+ base?: string;
19
+ output?: string;
20
+ names?: string;
21
+ strategy: string;
22
+ diffLevel: 'sentence' | 'word';
23
+ dryRun?: boolean;
24
+ sections?: boolean;
25
+ }
26
+
27
+ /**
28
+ * Register merge, conflicts, and merge-resolve commands with the program
29
+ */
30
+ export function register(program: Command): void {
31
+ // ==========================================================================
32
+ // MERGE command - Combine feedback from multiple reviewers (three-way merge)
33
+ // ==========================================================================
34
+
35
+ program
36
+ .command('merge')
37
+ .description('Merge feedback from multiple Word documents using three-way merge')
38
+ .argument('<docx...>', 'Word documents from reviewers')
39
+ .option('-b, --base <file>', 'Base document (original sent to reviewers). Auto-detected if not specified.')
40
+ .option('-o, --output <file>', 'Output file (default: writes to section files)')
41
+ .option('--names <names>', 'Reviewer names (comma-separated, in order of docx files)')
42
+ .option('--strategy <strategy>', 'Conflict resolution: first, latest, or interactive (default)', 'interactive')
43
+ .option('--diff-level <level>', 'Diff granularity: sentence or word (default: sentence)', 'sentence')
44
+ .option('--dry-run', 'Show conflicts without writing')
45
+ .option('--sections', 'Split merged output back to section files')
46
+ .action(async (docxFiles: string[], options: MergeOptions) => {
47
+ const {
48
+ mergeThreeWay,
49
+ formatConflict,
50
+ resolveConflict,
51
+ getBaseDocument,
52
+ checkBaseMatch,
53
+ saveConflicts,
54
+ } = await import('../merge.js');
55
+
56
+ // Validate reviewer files exist
57
+ for (const docx of docxFiles) {
58
+ if (!fs.existsSync(docx)) {
59
+ console.error(fmt.status('error', `Reviewer file not found: ${docx}`));
60
+ process.exit(1);
61
+ }
62
+ }
63
+
64
+ // Determine base document
65
+ let basePath = options.base;
66
+ let baseSource = 'specified';
67
+
68
+ if (!basePath) {
69
+ // Try to use .rev/base.docx
70
+ const projectDir = process.cwd();
71
+ basePath = getBaseDocument(projectDir) ?? undefined;
72
+
73
+ if (basePath) {
74
+ baseSource = 'auto (.rev/base.docx)';
75
+ } else {
76
+ console.log(chalk.yellow('\n No base document found in .rev/base.docx'));
77
+ console.log(chalk.dim(' Tip: Run "rev build docx" to automatically save the base document.\n'));
78
+ console.error(fmt.status('error', 'Base document required. Use --base <file.docx>'));
79
+ process.exit(1);
80
+ }
81
+ }
82
+
83
+ if (!fs.existsSync(basePath)) {
84
+ console.error(fmt.status('error', `Base document not found: ${basePath}`));
85
+ process.exit(1);
86
+ }
87
+
88
+ // Check similarity between base and reviewer docs
89
+ const { matches, similarity } = await checkBaseMatch(basePath, docxFiles[0]);
90
+ if (!matches) {
91
+ console.log(chalk.yellow(`\n Warning: Base document may not match reviewer file (${Math.round(similarity * 100)}% similar)`));
92
+ console.log(chalk.dim(' If this is wrong, use --base to specify the correct original document.\n'));
93
+ }
94
+
95
+ // Parse reviewer names
96
+ const names = options.names
97
+ ? options.names.split(',').map(n => n.trim())
98
+ : docxFiles.map((f, i) => {
99
+ // Try to extract name from filename (e.g., paper_reviewer_A.docx)
100
+ const basename = path.basename(f, '.docx');
101
+ const match = basename.match(/_([A-Za-z]+)$/);
102
+ return match ? match[1] : `Reviewer ${i + 1}`;
103
+ });
104
+
105
+ // Pad names if needed
106
+ while (names.length < docxFiles.length) {
107
+ names.push(`Reviewer ${names.length + 1}`);
108
+ }
109
+
110
+ const reviewerDocs = docxFiles.map((p, i) => ({
111
+ path: p,
112
+ name: names[i],
113
+ }));
114
+
115
+ console.log(fmt.header('Three-Way Merge'));
116
+ console.log();
117
+ console.log(chalk.dim(` Base: ${path.basename(basePath)} (${baseSource})`));
118
+ console.log(chalk.dim(` Reviewers: ${names.join(', ')}`));
119
+ console.log(chalk.dim(` Diff level: ${options.diffLevel}`));
120
+ console.log();
121
+
122
+ const spin = fmt.spinner('Analyzing changes...').start();
123
+
124
+ try {
125
+ const { merged, conflicts, stats, baseText } = await mergeThreeWay(basePath, reviewerDocs, {
126
+ diffLevel: options.diffLevel,
127
+ });
128
+
129
+ spin.stop();
130
+
131
+ // Display stats
132
+ console.log(fmt.table(['Metric', 'Count'], [
133
+ ['Total changes', stats.totalChanges.toString()],
134
+ ['Non-conflicting', stats.nonConflicting.toString()],
135
+ ['Conflicts', stats.conflicts.toString()],
136
+ ['Comments', stats.comments.toString()],
137
+ ]));
138
+ console.log();
139
+
140
+ let finalMerged = merged;
141
+
142
+ // Handle conflicts
143
+ if (conflicts.length > 0) {
144
+ console.log(chalk.yellow(`Found ${conflicts.length} conflict(s):\n`));
145
+
146
+ if (options.strategy === 'first') {
147
+ // Auto-resolve: take first reviewer's change
148
+ for (const conflict of conflicts) {
149
+ console.log(chalk.dim(` Conflict ${conflict.id}: using ${conflict.changes[0].reviewer}'s change`));
150
+ resolveConflict(conflict, 0);
151
+ }
152
+ } else if (options.strategy === 'latest') {
153
+ // Auto-resolve: take last reviewer's change
154
+ for (const conflict of conflicts) {
155
+ const lastIdx = conflict.changes.length - 1;
156
+ console.log(chalk.dim(` Conflict ${conflict.id}: using ${conflict.changes[lastIdx].reviewer}'s change`));
157
+ resolveConflict(conflict, lastIdx);
158
+ }
159
+ } else if (!options.dryRun) {
160
+ // Interactive resolution
161
+ for (let i = 0; i < conflicts.length; i++) {
162
+ const conflict = conflicts[i];
163
+ console.log(chalk.bold(`\nConflict ${i + 1}/${conflicts.length} (${conflict.id}):`));
164
+ console.log(formatConflict(conflict, baseText));
165
+ console.log();
166
+
167
+ const rl = readline.createInterface({
168
+ input: process.stdin,
169
+ output: process.stdout,
170
+ });
171
+
172
+ const answer = await new Promise<string>((resolve) =>
173
+ rl.question(chalk.cyan(` Choose (1-${conflict.changes.length}, s=skip): `), resolve)
174
+ );
175
+ rl.close();
176
+
177
+ if (answer.toLowerCase() !== 's' && !isNaN(parseInt(answer))) {
178
+ const choice = parseInt(answer) - 1;
179
+ if (choice >= 0 && choice < conflict.changes.length) {
180
+ resolveConflict(conflict, choice);
181
+ console.log(chalk.green(` ✓ Applied: ${conflict.changes[choice].reviewer}'s change`));
182
+ }
183
+ } else {
184
+ console.log(chalk.dim(' Skipped (will need manual resolution)'));
185
+ }
186
+ }
187
+ }
188
+
189
+ // Save unresolved conflicts for later
190
+ const unresolved = conflicts.filter(c => c.resolved === null);
191
+ if (unresolved.length > 0) {
192
+ saveConflicts(process.cwd(), conflicts, basePath);
193
+ console.log(chalk.yellow(`\n ${unresolved.length} unresolved conflict(s) saved to .rev/conflicts.json`));
194
+ console.log(chalk.dim(' Run "rev conflicts" to view, "rev merge-resolve" to resolve'));
195
+ }
196
+ }
197
+
198
+ // Write output
199
+ if (!options.dryRun) {
200
+ if (options.output) {
201
+ // Write to single file
202
+ fs.writeFileSync(options.output, finalMerged, 'utf-8');
203
+ console.log(fmt.status('success', `Merged output written to ${options.output}`));
204
+ } else if (options.sections) {
205
+ // Split to section files (TODO: implement section splitting)
206
+ console.log(chalk.yellow(' Section splitting not yet implemented'));
207
+ console.log(chalk.dim(' Use -o to specify output file'));
208
+ } else {
209
+ // Default: write to merged.md
210
+ const outPath = 'merged.md';
211
+ fs.writeFileSync(outPath, finalMerged, 'utf-8');
212
+ console.log(fmt.status('success', `Merged output written to ${outPath}`));
213
+ }
214
+
215
+ console.log();
216
+ console.log(chalk.dim('Next steps:'));
217
+ console.log(chalk.dim(' 1. rev review merged.md - Accept/reject changes'));
218
+ console.log(chalk.dim(' 2. rev comments merged.md - Address comments'));
219
+ if (conflicts.some(c => c.resolved === null)) {
220
+ console.log(chalk.dim(' 3. rev merge-resolve - Resolve remaining conflicts'));
221
+ }
222
+ } else {
223
+ console.log(fmt.status('info', 'Dry run - no output written'));
224
+ }
225
+ } catch (err) {
226
+ spin.stop();
227
+ const error = err as Error;
228
+ console.error(fmt.status('error', error.message));
229
+ if (process.env.DEBUG) console.error(error.stack);
230
+ process.exit(1);
231
+ }
232
+ });
233
+
234
+ // ==========================================================================
235
+ // CONFLICTS command - List unresolved conflicts
236
+ // ==========================================================================
237
+
238
+ program
239
+ .command('conflicts')
240
+ .description('List unresolved merge conflicts')
241
+ .action(async () => {
242
+ const { loadConflicts, formatConflict } = await import('../merge.js');
243
+ const projectDir = process.cwd();
244
+ const data = loadConflicts(projectDir);
245
+
246
+ if (!data) {
247
+ console.log(fmt.status('info', 'No conflicts file found'));
248
+ return;
249
+ }
250
+
251
+ const unresolved = data.conflicts.filter((c: any) => c.resolved === null);
252
+
253
+ if (unresolved.length === 0) {
254
+ console.log(fmt.status('success', 'All conflicts resolved!'));
255
+ return;
256
+ }
257
+
258
+ console.log(fmt.header(`Unresolved Conflicts (${unresolved.length})`));
259
+ console.log();
260
+ console.log(chalk.dim(` Base: ${data.base}`));
261
+ console.log(chalk.dim(` Merged: ${data.merged}`));
262
+ console.log();
263
+
264
+ for (const conflict of unresolved) {
265
+ console.log(chalk.bold(`Conflict ${conflict.id}:`));
266
+ // Show abbreviated info
267
+ console.log(chalk.dim(` Original: "${conflict.original.slice(0, 50)}${conflict.original.length > 50 ? '...' : ''}"`));
268
+ console.log(chalk.dim(` Options: ${conflict.changes.map((c: any) => c.reviewer).join(', ')}`));
269
+ console.log();
270
+ }
271
+
272
+ console.log(chalk.dim('Run "rev merge-resolve" to resolve conflicts interactively'));
273
+ });
274
+
275
+ // ==========================================================================
276
+ // MERGE-RESOLVE command - Interactively resolve merge conflicts
277
+ // ==========================================================================
278
+
279
+ program
280
+ .command('merge-resolve')
281
+ .alias('mresolve')
282
+ .description('Resolve merge conflicts interactively')
283
+ .option('--theirs', 'Accept all changes from last reviewer')
284
+ .option('--ours', 'Accept all changes from first reviewer')
285
+ .action(async (options: { theirs?: boolean; ours?: boolean }) => {
286
+ const { loadConflicts, saveConflicts, clearConflicts, resolveConflict, formatConflict } = await import('../merge.js');
287
+ const projectDir = process.cwd();
288
+ const data = loadConflicts(projectDir);
289
+
290
+ if (!data) {
291
+ console.log(fmt.status('info', 'No conflicts to resolve'));
292
+ return;
293
+ }
294
+
295
+ const unresolved = data.conflicts.filter((c: any) => c.resolved === null);
296
+
297
+ if (unresolved.length === 0) {
298
+ console.log(fmt.status('success', 'All conflicts already resolved!'));
299
+ clearConflicts(projectDir);
300
+ return;
301
+ }
302
+
303
+ console.log(fmt.header(`Resolving ${unresolved.length} Conflict(s)`));
304
+ console.log();
305
+
306
+ if (options.theirs) {
307
+ // Accept all from last reviewer
308
+ for (const conflict of unresolved) {
309
+ const lastIdx = conflict.changes.length - 1;
310
+ resolveConflict(conflict, lastIdx);
311
+ console.log(chalk.dim(` ${conflict.id}: accepted ${conflict.changes[lastIdx].reviewer}'s change`));
312
+ }
313
+ saveConflicts(projectDir, data.conflicts, data.base);
314
+ console.log(fmt.status('success', `Resolved ${unresolved.length} conflicts (--theirs)`));
315
+ } else if (options.ours) {
316
+ // Accept all from first reviewer
317
+ for (const conflict of unresolved) {
318
+ resolveConflict(conflict, 0);
319
+ console.log(chalk.dim(` ${conflict.id}: accepted ${conflict.changes[0].reviewer}'s change`));
320
+ }
321
+ saveConflicts(projectDir, data.conflicts, data.base);
322
+ console.log(fmt.status('success', `Resolved ${unresolved.length} conflicts (--ours)`));
323
+ } else {
324
+ // Interactive resolution
325
+ // Read base text for context display
326
+ let baseText = '';
327
+ try {
328
+ const { extractFromWord } = await import('../import.js');
329
+ const { text } = await extractFromWord(data.base);
330
+ baseText = text;
331
+ } catch {
332
+ // Can't read base, show without context
333
+ }
334
+
335
+ for (let i = 0; i < unresolved.length; i++) {
336
+ const conflict = unresolved[i];
337
+ console.log(chalk.bold(`\nConflict ${i + 1}/${unresolved.length} (${conflict.id}):`));
338
+ console.log(formatConflict(conflict, baseText));
339
+ console.log();
340
+
341
+ const rl = readline.createInterface({
342
+ input: process.stdin,
343
+ output: process.stdout,
344
+ });
345
+
346
+ const answer = await new Promise<string>((resolve) =>
347
+ rl.question(chalk.cyan(` Choose (1-${conflict.changes.length}, s=skip, q=quit): `), resolve)
348
+ );
349
+ rl.close();
350
+
351
+ if (answer.toLowerCase() === 'q') {
352
+ console.log(chalk.dim('\n Saving progress...'));
353
+ break;
354
+ }
355
+
356
+ if (answer.toLowerCase() !== 's' && !isNaN(parseInt(answer))) {
357
+ const choice = parseInt(answer) - 1;
358
+ if (choice >= 0 && choice < conflict.changes.length) {
359
+ resolveConflict(conflict, choice);
360
+ console.log(chalk.green(` ✓ Applied: ${conflict.changes[choice].reviewer}'s change`));
361
+ }
362
+ } else {
363
+ console.log(chalk.dim(' Skipped'));
364
+ }
365
+ }
366
+
367
+ saveConflicts(projectDir, data.conflicts, data.base);
368
+
369
+ const remaining = data.conflicts.filter((c: any) => c.resolved === null).length;
370
+ if (remaining === 0) {
371
+ console.log(fmt.status('success', '\nAll conflicts resolved!'));
372
+ clearConflicts(projectDir);
373
+ } else {
374
+ console.log(chalk.yellow(`\n ${remaining} conflict(s) remaining`));
375
+ }
376
+ }
377
+ });
378
+ }