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.

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