π¦ Hippo Memory Insights β 2026-05-17 #32775
Closed
Replies: 1 comment
-
|
This discussion has been marked as outdated by Daily Hippo Learn. A newer discussion is available at Discussion #32973. |
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
mem_56999cd476c3β Stale workflow.lock.ymlfiles cause CI churn; runmake recompileafter markdown workflow edits (score=0.994, verified, retrieved 10x)Top Memories Surfaced
mem_56999cd476c3(verified, score=0.994) β Stale.lock.ymlworkflow files cause recurring CI friction. Always runmake recompileafter editing markdown workflow files.mem_837ce85555f5(score=0.684) β Error reporting should include actionable suggestions, not just the error text.mem_7cfbbf7ec61e(score=0.690) β CIvalidate-yamljob scans all YAML files for ANSI escape sequences before other jobs run β embedding ANSI in YAML will silently break CI.mem_67941271277f(score=0.761) β Seescratchpad/validation-refactoring.mdfor step-by-step guidance on splitting complex validation files.mem_de1a9a9c2a51(score=0.662) β Runningmake agent-report-progresswith zero lint errors before everyreport_progresscall is mandatory; CI will fail if skipped.mem_d9d310064f1e(score=0.806) β Useskills/javascript-refactoring/SKILL.mdfor guidance when splitting large.cjsaction scripts.mem_8e1bfa8b3700(score=0.604) β Unformatted code causes immediate CI failures that block all other work; always runmake fmtbefore opening a PR.Suggested Improvements
Error Patterns
make build && make fmt(Checkpoint 1) +make agent-report-progresswith zero lint errors (Checkpoint 2) appears in many high-score memories but is still occasionally skipped, causing repeated failures.validate-yamlCI job catches these, but the root cause (tool output leaking into generated YAML) should be audited in any script that writes workflow files..lock.ymlfiles are the single highest-confidence incident pattern in the store. Enforcemake recompileas a mandatory post-step whenever a markdown workflow is edited.Code Quality
scratchpad/validation-refactoring.mddocuments a refactoring guide for large validation files. Files exceeding ~300 lines or mixing 2+ distinct domains are flagged as candidates for splitting.skills/javascript-refactoring/SKILL.mdrecommends splitting large.cjsfiles into smaller focused modules β worth reviewing any action script approaching 300 lines.JobName,StepID) prevent cross-domain misuse bugs. New code should prefer distinct named types over rawstring/int.CI Health
make test, >5 min) should stay in CI only β don't run locally. Usemake test-unit(~30s) during development.make deps-dev(~5 min) is a known slow step; cache it across runs if not already cached.make lint-cjsshould be run after any JavaScript action changes to avoid YAML-embedded lint failures.Quick Wins
make recompilereminder to PR template β the highest-confidence memory in the store (score=0.994) is about stale.lock.ymlfiles; a checklist item costs nothing.validate-yamlCI job already catches it, but a pre-commit hook would give faster feedback.scratchpad/validation-refactoring.mdin the developer guide β it exists but isn't prominently linked; adding a reference inDEVGUIDE.mdwould help.make fmtto git pre-commit hook β unformatted code is the most cited immediate CI failure; automating it locally eliminates the class.list_code_scanning_alertsusage in workflows β memorymem_1056edfadeacnotes that omittingstate: open+severity: critical,highproduces oversized MCP responses; audit all workflow prompts for this.Longer-term Themes
skills/) is a recurring theme β the store reflects that agents sometimes load skills preemptively rather than on-demand, wasting context. TheBE LAZYrule should be reinforced in agent onboarding docs..cjsfiles as technical debt. A periodic audit rule (e.g., CI warning when a.cjsexceeds 300 lines) would surface this automatically.Memory Health
View Memory Store Stats
hippo auditfor details) β worth pruning to keep the store sharp.References: Β§25984328816
Beta Was this translation helpful? Give feedback.
All reactions