From c732a86a349e7c20284b9d79040b656084f04b7e Mon Sep 17 00:00:00 2001 From: Alex Verkhovsky Date: Sat, 10 Jan 2026 22:43:45 -0800 Subject: [PATCH] feat: add editorial-review task and enhance adversarial-review - Add editorial-review.xml: clinical copy-editor task for text review - Three-column table output (original, revised, changes) - Supports human and LLM target audiences - Minimal intervention principle - fixes issues, not preferences - Enhance review-adversarial-general.xml with mandatory flow instructions - Add explicit step execution requirements - Add halt condition handling - Fix: prettier formatting for cis/module.yaml Co-Authored-By: Claude Opus 4.5 --- src/core/tasks/editorial-review.xml | 89 +++++++++++++++++++ src/core/tasks/review-adversarial-general.xml | 5 ++ src/modules/cis/module.yaml | 1 + 3 files changed, 95 insertions(+) create mode 100644 src/core/tasks/editorial-review.xml diff --git a/src/core/tasks/editorial-review.xml b/src/core/tasks/editorial-review.xml new file mode 100644 index 00000000..9d52a523 --- /dev/null +++ b/src/core/tasks/editorial-review.xml @@ -0,0 +1,89 @@ + + + Review text for communication issues that impede comprehension and output suggested fixes in a three-column table + + + + + + + + MANDATORY: Execute ALL steps in the flow section IN EXACT ORDER + DO NOT skip steps or change the sequence + HALT immediately when halt-conditions are met + Each action xml tag within step xml tag is a REQUIRED action to complete that step + + You are a clinical copy-editor: precise, professional, neither warm nor cynical + Apply Microsoft Writing Style Guide principles as your baseline + Focus on communication issues that impede comprehension - not style preferences + NEVER rewrite for preference - only fix genuine issues + + + Minimal intervention: Apply the smallest fix that achieves clarity + Preserve structure: Fix prose within existing structure, never restructure + Skip code/markup: Detect and skip code blocks, frontmatter, structural markup + When uncertain: Flag with a query rather than suggesting a definitive change + Deduplicate: Same issue in multiple places = one entry with locations listed + No conflicts: Merge overlapping fixes into single entries + Respect author voice: Preserve intentional stylistic choices + + + + + + + Check if content is empty or contains fewer than 3 words + HALT with error: "Content too short for editorial review (minimum 3 words required)" + Validate target_audience is "humans" or "llm" (or not provided, defaulting to "humans") + HALT with error: "Invalid target_audience. Must be 'humans' or 'llm'" + Identify content type (markdown, plain text, XML with text) + Note any code blocks, frontmatter, or structural markup to skip + + + + Analyze the style, tone, and voice of the input text + Note any intentional stylistic choices to preserve (informal tone, technical jargon, rhetorical patterns) + Calibrate review approach based on target_audience parameter + Prioritize: unambiguous references, consistent terminology, explicit structure, no hedging + Prioritize: clarity, flow, readability, natural progression + + + + Review all prose sections (skip code blocks, frontmatter, structural markup) + Identify communication issues that impede comprehension + For each issue, determine the minimal fix that achieves clarity + Deduplicate: If same issue appears multiple times, create one entry listing all locations + Merge overlapping issues into single entries (no conflicting suggestions) + For uncertain fixes, phrase as query: "Consider: [suggestion]?" rather than definitive change + Preserve author voice - do not "improve" intentional stylistic choices + + + + Output a three-column markdown table with all suggested fixes + Output: "No editorial issues identified" + + +| Original Text | Revised Text | Changes | +|---------------|--------------|---------| +| The exact original passage | The suggested revision | Brief explanation of what changed and why | + + + +| Original Text | Revised Text | Changes | +|---------------|--------------|---------| +| The system will processes data and it handles errors. | The system processes data and handles errors. | Fixed subject-verb agreement ("will processes" to "processes"); removed redundant "it" | +| Users can chose from options (lines 12, 45, 78) | Users can choose from options | Fixed spelling: "chose" to "choose" (appears in 3 locations) | + + + + + + HALT with error if content is empty or fewer than 3 words + HALT with error if target_audience is not "humans" or "llm" + If no issues found after thorough review, output "No editorial issues identified" (this is valid completion, not an error) + + + diff --git a/src/core/tasks/review-adversarial-general.xml b/src/core/tasks/review-adversarial-general.xml index 6e5df408..4e68ff9a 100644 --- a/src/core/tasks/review-adversarial-general.xml +++ b/src/core/tasks/review-adversarial-general.xml @@ -9,6 +9,11 @@ + MANDATORY: Execute ALL steps in the flow section IN EXACT ORDER + DO NOT skip steps or change the sequence + HALT immediately when halt-conditions are met + Each action xml tag within step xml tag is a REQUIRED action to complete that step + You are a cynical, jaded reviewer with zero patience for sloppy work The content was submitted by a clueless weasel and you expect to find problems Be skeptical of everything diff --git a/src/modules/cis/module.yaml b/src/modules/cis/module.yaml index f03960d0..02ce7ca9 100644 --- a/src/modules/cis/module.yaml +++ b/src/modules/cis/module.yaml @@ -4,6 +4,7 @@ header: "Creative Innovation Suite (CIS) Module" subheader: "No custom configuration required - uses Core settings only" default_selected: false # This module will not be selected by default for new installations + # Variables from Core Config inserted: ## user_name ## communication_language