Skip to content

feat(engineer-brain): show categorized BRAIN.md diff on update - #77

Merged
Hrithik-Gavankar merged 2 commits into
Hrithik-Gavankar:mainfrom
Rutujakirad:feat/14-brain-update-diff
Sep 29, 2026
Merged

Hrithik-Gavankar merged 2 commits into
Hrithik-Gavankar:mainfrom
Rutujakirad:feat/14-brain-update-diff

Conversation

@Rutujakirad

Copy link
Copy Markdown

Summary

Adds diff mode for engineer-brain update (#14): after refreshing BRAIN.md, show a dated, categorized learning summary and a colored before/after diff so users can see what the brain learned. Also adds --quiet, local history/ snapshots, and engineer-brain diff [days-ago].

Type of Change

  • New feature

Changes Made

  • Add core/scripts/brain-history.sh (begin / finish / diff) for snapshot + categorized colored diff
  • Wire into Cursor SKILL.md, COMMANDS.md, and install.sh
  • Add smoke test + CI step; update README command table

Testing

  • Ran bash tests/brain-history/smoke.sh
  • Manual begin → edit → finish / --quiet / diff 0
  • Verified install script copies brain-history.sh
  • Tested Cursor skill wiring

Checklist

  • Changes follow existing code style
  • Documentation updated (COMMANDS.md, Cursor SKILL.md, README)
  • Tested with Cursor platform
  • Commit message follows conventional commits

Related Issues

Closes #14

After update, snapshot history and print a dated learning summary
plus a colored unified diff. Adds diff [days-ago], --quiet,
install/CI wiring, and docs for Hrithik-Gavankar#14.
Comment thread core/scripts/brain-history.sh Resolved
Comment thread core/scripts/brain-history.sh Resolved
@Hrithik-Gavankar
Hrithik-Gavankar self-requested a review September 29, 2026 20:34
@Hrithik-Gavankar

Copy link
Copy Markdown
Owner

@Rutujakirad
Re-checked f36e0a6 — both major items look good (strict velocity + table emphasis normalization, smoke covers both). Approving; thank you for the contribution 😁

@Hrithik-Gavankar Hrithik-Gavankar left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM!

@Hrithik-Gavankar
Hrithik-Gavankar merged commit 364b656 into Hrithik-Gavankar:main Sep 29, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Feature update or enhancement feat New Feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Diff mode for update — show what changed in BRAIN.md

2 participants