You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Highest-confidence memory: mem_05f7e96e10bc β stale .lock.yml workflow files causing CI churn (score=1.00, verified, retrieved 2Γ)
Top Memories Surfaced
mem_05f7e96e10bc(verified, score=1.00, retrieved 2Γ) β Stale workflow .lock.yml files cause CI churn; always run make recompile after editing markdown workflow files.
mem_5b74c0634f76(verified, score=1.00, retrieved 1Γ) β node: command not found on GPU/self-hosted paths when node runtime is missing; validate runtimes and PATH early.
mem_52c8036cdb72(score=1.00) β validate-yaml CI job scans all YAML for ANSI escape sequences before other jobs run β do not embed raw escape codes in YAML.
mem_0238ab5efffb(score=0.95) β CI will fail if make fmt is skipped before opening a PR; unformatted code blocks all other work.
mem_9edf9cfd73b3(score=0.84) β Skipping make fmt has caused 5 CI failures in a single day β always format before committing.
mem_b8ccefa844ba(score=1.00) β JavaScript refactoring guide: skills/javascript-refactoring/SKILL.md β split CJS logic into separate files.
mem_5822048d553a(score=0.83) β File splitting decision tree: split when >300 lines or when β₯2 distinct domains are present.
Suggested Improvements
Error Patterns
Stale .lock.yml files (mem_05f7e96e10bc): The highest-confidence recurring incident. After any markdown workflow edit, run make recompile. Consider adding a pre-commit hook or CI lint step that detects out-of-sync .lock.yml files automatically.
Missing node on self-hosted runners (mem_5b74c0634f76): Validate that runtimes.node is declared in workflow YAML and that node is on PATH before any JS tooling is invoked. A startup health-check step in .github/aw/ would catch this early.
Code Quality
JavaScript file fragmentation: skills/javascript-refactoring/SKILL.md and pkg/workflow/js/ guidance is in memory but may not be enforced. Consider a lint rule or convention check that flags .cjs files exceeding 300 lines.
Type-safety patterns: The store contains strong guidance on using distinct string types (e.g., JobName vs StepID) to prevent mix-up bugs. Worth a code-review pass on new workflow-parsing code in internal/.
Validation complexity: scratchpad/validation-refactoring.md documents a step-by-step guide β ensure it stays current as validation logic evolves.
CI Health
make fmt gate: CI failures from formatting are the most repeated pattern (5 in one day). A mandatory make fmt --check step before any merge (or a pre-push hook) would eliminate this class entirely.
YAML ANSI escape validation: The validate-yaml job is a good gate but only runs in CI. Add a local make lint-yaml target so agents catch this before pushing.
make recompile reminder: Workflow lock files drifting out of sync is flagged as a recurring incident. A CI check comparing the committed .lock.yml to a freshly compiled output would make this self-healing.
Quick Wins
Add make fmt --check to pre-push hook β eliminates the rejig docsΒ #1 source of CI failures with ~2 min effort.
Add make recompile diff check to CI β catches stale .lock.yml automatically; prevents the recurring incident.
Local make lint-yaml target β mirrors the validate-yaml CI job so contributors get fast local feedback.
Node PATH validation in workflow startup β one shell check (command -v node || exit 1) prevents the self-hosted node: not found class of failures.
Prune 7 low-quality memories β run hippo audit and remove stale/low-signal entries to keep recall precision high.
Longer-term Themes
Formatting discipline: Multiple memories (score β₯ 0.84) converge on the same message β formatting must happen before commit, not after. This suggests the team would benefit from integrated tooling (e.g., editor format-on-save + CI gate) rather than relying on memory/documentation alone.
Workflow recompilation lifecycle: Two verified incidents relate to workflow files being edited without recompilation. This is a systemic process gap that merits a dedicated improvement (ADR, tooling, or CI enforcement).
JavaScript modularization: Several memories reference the JS refactoring skill and file-splitting guidelines, suggesting ongoing tension between file size and modular structure in pkg/workflow/js/. A focused refactoring session is warranted.
Memory Health
Metric
Count
Total memories
525
Episodic
473
Semantic (consolidated)
52
At risk (<0.2 strength)
0
Low-quality flagged
7
Duplicates removed this run
25
Avg strength
1.00
Low-quality memories to review
Run mcpscripts hippo --args "audit" to get the full list of 7 low-quality memories flagged during the consolidation pass. These are candidates for pruning to keep recall precision high.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
Summary
mem_05f7e96e10bcβ stale.lock.ymlworkflow files causing CI churn (score=1.00, verified, retrieved 2Γ)Top Memories Surfaced
mem_05f7e96e10bc(verified, score=1.00, retrieved 2Γ) β Stale workflow.lock.ymlfiles cause CI churn; always runmake recompileafter editing markdown workflow files.mem_5b74c0634f76(verified, score=1.00, retrieved 1Γ) βnode: command not foundon GPU/self-hosted paths when node runtime is missing; validate runtimes andPATHearly.mem_52c8036cdb72(score=1.00) βvalidate-yamlCI job scans all YAML for ANSI escape sequences before other jobs run β do not embed raw escape codes in YAML.mem_0238ab5efffb(score=0.95) β CI will fail ifmake fmtis skipped before opening a PR; unformatted code blocks all other work.mem_9edf9cfd73b3(score=0.84) β Skippingmake fmthas caused 5 CI failures in a single day β always format before committing.mem_b8ccefa844ba(score=1.00) β JavaScript refactoring guide:skills/javascript-refactoring/SKILL.mdβ split CJS logic into separate files.mem_5822048d553a(score=0.83) β File splitting decision tree: split when >300 lines or when β₯2 distinct domains are present.Suggested Improvements
Error Patterns
.lock.ymlfiles (mem_05f7e96e10bc): The highest-confidence recurring incident. After any markdown workflow edit, runmake recompile. Consider adding a pre-commit hook or CI lint step that detects out-of-sync.lock.ymlfiles automatically.nodeon self-hosted runners (mem_5b74c0634f76): Validate thatruntimes.nodeis declared in workflow YAML and thatnodeis onPATHbefore any JS tooling is invoked. A startup health-check step in.github/aw/would catch this early.Code Quality
skills/javascript-refactoring/SKILL.mdandpkg/workflow/js/guidance is in memory but may not be enforced. Consider a lint rule or convention check that flags.cjsfiles exceeding 300 lines.JobNamevsStepID) to prevent mix-up bugs. Worth a code-review pass on new workflow-parsing code ininternal/.scratchpad/validation-refactoring.mddocuments a step-by-step guide β ensure it stays current as validation logic evolves.CI Health
make fmtgate: CI failures from formatting are the most repeated pattern (5 in one day). A mandatorymake fmt --checkstep before any merge (or a pre-push hook) would eliminate this class entirely.validate-yamljob is a good gate but only runs in CI. Add a localmake lint-yamltarget so agents catch this before pushing.make recompilereminder: Workflow lock files drifting out of sync is flagged as a recurring incident. A CI check comparing the committed.lock.ymlto a freshly compiled output would make this self-healing.Quick Wins
make fmt --checkto pre-push hook β eliminates the rejig docsΒ #1 source of CI failures with ~2 min effort.make recompilediff check to CI β catches stale.lock.ymlautomatically; prevents the recurring incident.make lint-yamltarget β mirrors thevalidate-yamlCI job so contributors get fast local feedback.command -v node || exit 1) prevents the self-hostednode: not foundclass of failures.hippo auditand remove stale/low-signal entries to keep recall precision high.Longer-term Themes
pkg/workflow/js/. A focused refactoring session is warranted.Memory Health
Low-quality memories to review
Run
mcpscripts hippo --args "audit"to get the full list of 7 low-quality memories flagged during the consolidation pass. These are candidates for pruning to keep recall precision high.References: Β§25541505618
Beta Was this translation helpful? Give feedback.
All reactions