Enable the builtin /remember skill

[bl-ue]

Summary by CodeRabbit

• New Features

  • Added a built-in "/remember" skill to review session memory and manage session notes (can be enabled by users).

• Settings & Defaults

  • Line numbers are now displayed by default.
  • New toggle: "Enable remember skill" (default: off) exposed in Misc settings.

• Documentation

  • README updated to list "Session memory" and the "/remember" skill.

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds a new optional "/remember" built-in skill: config flag, types, UI toggle, patch integration to inject the skill registration into target code, changelog/readme updates, and a new patch writer module that implements the injection logic.

Changes

Cohort / File(s)	Summary
Changelog & Docs CHANGELOG.md, README.md	Inserted Unreleased changelog entry for the /remember skill and added README mention under Swarm mode features.
Default settings src/defaultSettings.ts	Changed misc.suppressLineNumbers from true → false and added new misc.enableRememberSkill: false.
Types src/types.ts	Added enableRememberSkill: boolean to MiscConfig.
Patch integration src/patches/index.ts	Imported and registered new remember-skill patch in PATCH_DEFINITIONS and patch implementations; patch application gated by config.settings.misc?.enableRememberSkill.
Patch implementation src/patches/rememberSkill.ts	New module exporting `writeRememberSkill(oldFile: string): string
UI src/ui/components/MiscView.tsx	Added "Enable remember skill" toggle bound to settings.misc.enableRememberSkill (default false) with standard getValue/toggle handling.

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant UI as MiscView
  participant PatchRunner as patches/index
  participant Writer as rememberSkill.ts
  participant Codebase

  User->>UI: Toggle "Enable remember skill" ON
  UI->>PatchRunner: Persist setting in config
  PatchRunner->>Writer: call writeRememberSkill(oldFile)
  Writer->>Codebase: analyze file, inject registration block
  Writer-->>PatchRunner: return patched content / null
  PatchRunner->>Codebase: write patched file (if returned)

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

• Don't suppress line numbers by default #360 — modifies suppressLineNumbers default and related UI/settings handling (overlaps with src/defaultSettings.ts and MiscView.tsx changes).
• Misc fixes #440 — changes to src/patches/index.ts and patch definitions/metadata (related to adding the remember-skill patch).

Poem

🐰 I sniffed the code and found a clue,
A tiny skill to help recall for you.
Toggle it on, let patches play,
Memories gathered — hip hooray! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: enabling a built-in /remember skill. The changeset implements this feature across multiple files including settings, UI, patches, and documentation.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch enable-remember-skill

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@src/defaultSettings.ts`:
- Line 709: DEFAULT_SETTINGS.misc.enableRememberSkill is true but MiscView.tsx
defines defaultMisc.enableRememberSkill as false; make these defaults consistent
by updating the mismatch: either set DEFAULT_SETTINGS.misc.enableRememberSkill
to false or change defaultMisc.enableRememberSkill in MiscView.tsx to true so
both DEFAULT_SETTINGS and defaultMisc use the same boolean, and ensure any tests
or UI labels that rely on enableRememberSkill reflect the corrected default.

🧹 Nitpick comments (1)

src/patches/rememberSkill.ts (1)

42-52: Error handling is adequate and follows codebase conventions.

The regex pattern correctly targets the documented minified structure, and the generic error message is consistent with other patches in the codebase. The detailed code comments (lines 14-20) already explain the expected minified format. If desired, decomposing the pattern into separate checks could provide more granular error reporting, but this is not necessary for functionality.

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@README.md`:
- Around line 108-112: The README TOC contains "Session memory" and "/remember
skill" anchors that don't match any section headings, causing broken links;
either add corresponding headings (e.g., create "## Session memory" and "##
/remember skill" sections with content so the anchors resolve) or remove those
two TOC lines until the sections exist; update the TOC entries to match the
exact heading text/slug used by the markdown renderer so the anchors are valid.