Skip to content

runner.rs is the last file over the 500-code-line bar (677 non-test code lines) #5

Description

@JoshuaBearup

Housekeeping observation, zero urgency. CLAUDE.md sets a 500-code-line ceiling (excluding tests/comments/whitespace). At fd1c587, crates/norn/src/loop/runner.rs sits at 677 non-test code lines (lines 1-914, before the #[cfg(test)] module) — the only production file in the workspace still over the bar; everything else above 500 is test code or reference/ material. The rest of loop/ already shows the carve-out pattern (helpers/iteration/tool_dispatch/linger/compaction/inflight_compaction), so this is presumably just the one that has not had its turn.

Optional fix (take it or leave it): the natural seams looked like (a) the assistant-message / session-event assembly around lines 484-520, (b) the schema-enforcement/retry block, and (c) the post-LLM hook plumbing — any one of those moving to a sibling module would clear the bar. Entirely your call on timing, given Wave 3 is actively landing in this file.


Surfaced during a documentation/quality review of norn for the Meridian workspace; line count verified against fd1c587. Optional fix only.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions