Skip to content

feat: add feedback command #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
fmueller merged 7 commits into from
Jan 22, 2026
+1,676 −27
Merged

feat: add feedback command #13

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c0a97e0
feat: add feedback command
fmueller Dec 31, 2025
78d6647
refactor: extract shared utilities and add --seed/--top-p to feedback
fmueller Jan 22, 2026
aa61a6c
docs: add feedback command to CHANGELOG
fmueller Jan 22, 2026
5b50ce6
docs: add category definitions to feedback prompt
fmueller Jan 22, 2026
3c367ee
feat: improve feedback prompt clarity and focus handling
fmueller Jan 22, 2026
af5bd62
feat: expand feedback category definitions for better LLM guidance
fmueller Jan 22, 2026
55af1ba
feat: strip emojis from feedback report fields
fmueller Jan 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Added

- `--seed` and `--top-p` CLI options for all LLM commands (`idea`, `brief`, `write`, `meta`) to control output reproducibility
- `feedback` command for reviewing drafts against SEO briefs without rewriting
- Outputs structured reports (Markdown, JSON, or both) with issues, strengths, and actionable checklist
- Supports `--section` to limit review to specific outline sections
- Supports `--focus` to narrow review scope (seo, structure, clarity, style, evidence)
- Optional `--note` for grounding feedback with source material
- `--seed` and `--top-p` CLI options for all LLM commands (`idea`, `brief`, `write`, `meta`, `feedback`) to control output reproducibility
- `--postedit-seed` and `--postedit-top-p` CLI options for the `translate` command's LLM post-edit pass

## 0.1.0 - 2025-12-29
Expand Down
5 changes: 1 addition & 4 deletions src/scribae/brief.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
from pydantic_ai.settings import ModelSettings

from .idea import Idea, IdeaList
from .io_utils import NoteDetails, load_note
from .io_utils import NoteDetails, Reporter, load_note
from .language import LanguageMismatchError, LanguageResolutionError, ensure_language_output, resolve_output_language
from .llm import LLM_OUTPUT_RETRIES, LLM_TIMEOUT_SECONDS, OpenAISettings, apply_optional_settings, make_model
from .project import ProjectConfig
Expand Down Expand Up @@ -142,9 +142,6 @@ class BriefingContext:
language: str


Reporter = Callable[[str], None] | None


def prepare_context(
*,
note_path: Path,
Expand Down
Loading