π¦ Hippo Memory Insights β 2026-05-16 #32584
Closed
Replies: 1 comment
-
|
This discussion has been marked as outdated by Daily Hippo Learn. A newer discussion is available at Discussion #32775. |
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β "Recurring incident: stale workflow .lock.yml files cause churn and CI friction; runmake recompileafter markdown workflow edits." (score=1.000, verified)Top Memories Surfaced
mem_56999cd476c3β Stale.lock.ymlfiles from not runningmake recompileafter editing markdown workflows is the top recurring incident causing CI friction and churn.mem_d9d310064f1eβ JavaScript refactoring skill (skills/javascript-refactoring/SKILL.md) guides splitting large.cjsfiles; high relevance for maintainability work.mem_67941271277fβscratchpad/validation-refactoring.mddocuments a step-by-step refactoring guide for validation complexity; flagged multiple times.mem_7cfbbf7ec61eβ Thevalidate-yamlCI job scans all YAML files for ANSI escape sequences before other jobs β breaking this causes an early hard stop.mem_de1a9a9c2a51β Skippingmake fmtbefore committing causes immediate CI failures; flagged "CI WILL FAIL β this is automatic and non-negotiable."mem_837ce85555f5β Error reporting should include suggestions (help-text-standards skill); a recurring quality expectation across the codebase.mem_05ab47a37d26β Architecture uses runtime file copying instead of embedded scripts; relevant for anyone adding new JS/shell script assets.Suggested Improvements
Error Patterns
.lock.ymlfiles (top verified incident,mem_56999cd476c3): Agents and contributors regularly forget to runmake recompileafter editing.mdworkflow files. Consider adding a CI check or pre-commit hook that detects.lock.ymlfiles out of sync with their source.md.mem_8e1bfa8b3700,mem_de1a9a9c2a51): This has caused 5 CI failures in a single day. A pre-commit hook runningmake fmtwould eliminate this entirely.mem_7cfbbf7ec61e): Thevalidate-yamljob is an early hard stop. Agents should strip ANSI before writing descriptions/comments to compiled YAML β already handled inpkg/workflow/compiler_yaml.goviastringutil.StripANSI(), but awareness should be reinforced.Code Quality
mem_67941271277f):scratchpad/validation-refactoring.mddocuments known refactoring work. This should be progressed β files >300 lines containing 2+ domains are candidates for splitting per the decision tree inmem_f08660df2e11..cjsfile size (mem_d9d310064f1e): The JS refactoring skill exists for a reason β large files in GitHub Actions JS steps should be split using thejavascript-refactoringskill guidance.mem_fa8597c52cf4,mem_1c3f75657068): New code should prefer typed fields (e.g.,JobNamevs plainstring) for compile-time safety. Use distinct types to prevent cross-type assignment bugs.CI Health
make recompilediscipline: The rejig docsΒ #1 CI friction source. Automate or enforce at PR open time.make fmtbefore every commit: Non-negotiable. Should be a pre-commit hook, not a developer memory tax.make test) is slow (>5 min): Let CI run it; don't block local development on it. Usemake test-unit(~3 min) locally.Quick Wins
make fmtβ eliminates the single most common CI failure category. Simple to add to.git/hooks/pre-commitor document in DEVGUIDE.md..lock.ymlfiles β detect when.mdsource is newer than its compiled.lock.yml. Prevents the top verified incident.scratchpad/validation-refactoring.mdβ turn the documented plan into actual file splits; the groundwork is already laid.make recompilemore prominently in DEVGUIDE.md β current awareness is low given the incident rate.hippo auditβ 7 low-quality memories were flagged this cycle; pruning them keeps recall sharp.Longer-term Themes
error-messages,javascript-refactoring, anderror-recovery-patternsexist but are not consistently invoked. A skill-invocation audit across recent PRs may reveal patterns.make recompileincident pattern and thevalidate-yamlhard stop together suggest that the compilation pipeline deserves its own runbook entry alongside the existingworkflow-health.md.Memory Health
hippo auditfor details and pruning candidates)View consolidation details
References:
Beta Was this translation helpful? Give feedback.
All reactions