Skip to content

[deep-report] Add make fmt pre-commit hook to eliminate top CI failure class #30374

@github-actions

Description

@github-actions

Description

The #1 verified incident in the Hippo Memory store (score 1.00, verified, 525 memories) is: "Unformatted code causes immediate CI failures — make fmt is mandatory before commit/PR." Despite CI enforcement, this issue recurs because there is no pre-commit barrier. Adding a git pre-commit hook that runs make fmt and refuses commits that would fail CI formatting checks would eliminate this entire class of failures at source.

Expected Impact

  • Eliminates the most common class of CI failures (formatting)
  • Removes developer friction from the red/green/re-push cycle
  • Reduces wasted CI minutes on trivial failures
  • Complementary to the existing CI gate (defence in depth)

Implementation

  1. Add .githooks/pre-commit script:
    #!/bin/sh
    make fmt
    git add -u  # re-stage formatted changes
  2. Add make setup-hooks target to Makefile that runs git config core.hooksPath .githooks
  3. Document in DEVGUIDE.md: run make setup-hooks after cloning

Alternatively, use a lefthook.yml or .pre-commit-config.yaml if those tools are already present in the repo.

Suggested Agent

Copilot coding agent — Makefile + hook script creation, documentation update.

Estimated Effort

Quick (< 1 hour)

Data Source

DeepReport Intelligence Briefing — 2026-05-05 · Hippo Memory Insights §25362493917

Generated by DeepReport - Intelligence Gathering Agent · ● 369.8K ·

  • expires on May 7, 2026, 3:34 PM UTC

Metadata

Metadata

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions