π¦ Hippo Memory Insights β 2026-05-30 #35885
Closed
Replies: 1 comment
-
|
This discussion has been marked as outdated by Daily Hippo Learn. A newer discussion is available at Discussion #36081. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Summary
Top Memories Surfaced
gh-awis a GitHub CLI extension (gh aw) β not the GitHub Copilot CLI. Do not conflate the two when writing workflow prompts. (tag: important-note-gh-aw-vs-github-copilot-cli)list_code_scanning_alertscalls: always includestate: openandseverity: critical,highto avoid oversized MCP responses. (tag: checkpoint-2)make build && make recompileβ rebuilds binary and recompiles all workflow lock files. (tag: merging-main-branch)make build && make fmtimmediately after the first substantial code edit. Cheap (~2s), gives immediate feedback. (tag: checkpoint-1)report_progress/PR. (tag: mandatory-pre-commit-and-pre-pr-validation)make fmtis non-negotiable before committing..lock.ymlfiles are the top recurring CI incident: always runmake recompileafter any markdown workflow change. (score 0.950 β highest CI signal)Suggested Improvements
Error Patterns
make recompilestep in the developer workflow could prevent this automatically.make fmtbefore commits has caused 5+ CI failures in a single day. Consider enforcing format checks in a Git pre-commit hook or adding a CI fast-fail job.list_code_scanning_alertswithout filters cause agent failures. Workflow templates should includestate: open+severity: critical,highguards by default.Code Quality
scratchpad/validation-refactoring.mddocuments a step-by-step guide for splitting validators. Validators exceeding 300 lines or containing 2+ unrelated domains should be split.skills/javascript-refactoring/SKILL.mdhas been surfaced repeatedly for JS splitting work inactions/setup/js/. Files there should be kept under 300 lines.skills/developer/SKILL.mdcovers the full coding conventions β agents should reference it when unclear about validation architecture or file naming.CI Health
.lock.ymlfiles after markdown changes β addmake recompileto the contribution checklist and/or as a CI gate.validate-yamljob in.github/workflows/ci.ymlscans for ANSI escape sequences β ensure no debug output leaks into YAML/lock files.make test-unittakes ~3 minutes. Prefer selective testing (go test ./pkg/specific/...) during development; let CI run the full suite.Quick Wins
make recompileβ directly targets the rejig docsΒ #1 recurring CI incident (stale lock files).make fmtβ prevents the unformatted-code CI failures that have caused 5 failures/day.mcpscripts hippo --args "audit"and prune stale entries to keep the store precise.state: open+severity: critical,highto alllist_code_scanning_alertsworkflow uses β prevents oversized MCP payloads.Longer-Term Themes
pkg/parser/validators against the 300-line rule would reduce long-term debt.actions/setup/js/β/tmp/gh-aw/actions): This pattern has generated confusion. Better documentation or a diagram of the copy flow would reduce agent mistakes.Memory Health
View Memory Store Details
Pruning candidates: 52 memories with strength < 0.2, mostly cluster around the
checkpoint-2tag group β these are older episodic variants that have been superseded by consolidated semantic memories. Runningmcpscripts hippo --args "audit"will identify which to drop.References:
Beta Was this translation helpful? Give feedback.
All reactions