Skip to content

architecture-guardrails red on main: backend/utils/llm grew to 35 files without a package map #9886

Description

@krushnarout

PR #8501 (commit 147ef07, fix(backend): ground proactive insights and memory extraction in the current date) added backend/utils/llm/temporal.py, growing the package from its grandfathered 34 source files to 35 without adding a package-root ARCHITECTURE.md/README.md or updating .github/scripts/package_architecture_baseline.json.

As a result scripts/pr-preflight / the pre-push hook's repo-wide architecture-guardrails check now fails for every branch based on current main, even app-only diffs:

::error file=backend/utils/llm,line=1,title=Package architecture map required::backend/utils/llm grew from its grandfathered 34 source files to 35 without a package-root ARCHITECTURE.md or README.md. Add a map or revert the growth. See https://github.com/BasedHardware/omi/issues/9446

Reproduce on a clean checkout of main: python3 .github/scripts/check_arch_guardrails.py --base origin/main --changed-files app/lib/pages/chat/page.dart (any changed file selects the repo-wide ratchet).

Fix is either an ARCHITECTURE.md for backend/utils/llm (preferred per #9446) or a baseline bump. Also worth checking why the check didn't block #8501 in CI.

🤖 Generated with Claude Code

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Fields

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