diff --git a/.claude/skills/commit/skill.md b/.claude/skills/commit/skill.md index 7dac70c..5ae1963 100644 --- a/.claude/skills/commit/skill.md +++ b/.claude/skills/commit/skill.md @@ -1,6 +1,6 @@ --- name: "commit" -description: "Safe commit with pre-audit. Use /commit instead of git commit directly. Scans staged changes for César-specific violations, runs the verification loop (test → lint), then creates a properly formatted commit." +description: "Safe commit with pre-audit. Use /commit instead of git commit directly. Scans staged changes for César-specific violations, runs the verification loop (test → lint), updates docs (CHANGELOG, ROADMAP, README, CLAUDE.md), then creates a properly formatted commit." user_invocable: true --- @@ -115,6 +115,8 @@ If permissions were added, ask the user to confirm they are necessary — Chrome **If `--fast` was passed**: skip this step entirely and go to Step 5. +> Note: `--fast` skips tests/lint only — Step 5 (doc sync) is never skipped. + Run these in order. Stop immediately if any fails. ```sh @@ -129,7 +131,20 @@ If any command fails, show the error output and stop. Fix the issue before commi --- -## Step 5 — Create the commit +## Step 5 — Update documentation and roadmap + +Before committing, check whether the staged changes affect anything documented in the project's markdown files. Review and update as needed: + +1. **CHANGELOG.md** — Add an entry under the current version section (or create a new `[Unreleased]` section) describing what changed. Follow the existing Keep a Changelog format (Added / Fixed / Changed / Removed). +2. **ROADMAP.md** — If the commit ships, progresses, or completes a roadmap item, update its status (e.g., 📋 → 🔨 or 🔨 → ✅). If the commit introduces a new planned feature not yet on the roadmap, add it. +3. **README.md** — If the commit changes user-facing behavior, UI, supported LLMs, or anything mentioned in the README, update the relevant section. +4. **CLAUDE.md** — If the commit changes architecture, build setup, key constraints, or commands, update the dev guide. + +Only touch files that actually need updating — don't add noise. Stage the updated docs together with the code changes. + +--- + +## Step 6 — Create the commit Stage specific files (never `git add -A` or `git add .`): diff --git a/CHANGELOG.md b/CHANGELOG.md index 13f92a4..2725a3c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -14,6 +14,8 @@ All notable changes to César are documented here. - Bundle size reporting in build output (metafile) ### Fixed +- Comment prefill polls for editor (up to 2s) instead of fixed 500ms delay — fixes race on slow connections +- Comment prefill scopes broader search to same post ancestor — prevents injecting into wrong post - `escapeHTML` rewritten as pure string replacement — works in service workers (no `document` dependency) - `chrome.storage` wrappers now properly reject on `runtime.lastError` - LLM error handlers use `await` instead of `return` (errors were silently swallowed) @@ -22,6 +24,22 @@ All notable changes to César are documented here. - Removed stale `reason_fr` fallbacks from overlay manager and feed scanner ### Changed +- Overlay dismiss animates out (scale + fade) instead of `display: none` — smoother UX +- Inline styles replaced with CSS classes across overlay manager and popup (no more `style.color` in JS) +- All `!important` rules removed from overlay CSS — specificity fixed via selector weight +- Text arrows (`▼`) replaced with inline SVGs for cross-platform consistency +- AbortController wired to all overlay event listeners for proper cleanup on re-inject +- Overlay CSS resets font properties on `*` to prevent LinkedIn style bleed +- Popup: `html lang` changed from `fr` to `en`; scrollbar styled; `max-height` on reveal panel reduced +- Minimum font sizes raised to 11px across badges, labels, and footer for readability +- Low-contrast text colors bumped (`#606078` → `#9a9ab0`, `#3a3a4e` → `#9a9ab0`) +- Added `aria-label`, `aria-expanded`, `aria-controls`, and proper `