Skip to content

[flows-tinyflows] flows+cron: split oversized KEEP files #67

Closed

Description

@senamakel

Moved from tinyhumansai#8045, which is closed in favour of this issue. Part of tinyhumansai#8055.

Mechanical splits, no behaviour change:

  • cron/scheduler.rs (505) → scheduler/{dispatcher,live_agents,manual_run}.rs.
  • cron/schemas.rs (492) → declarations vs handlers.
  • flows/schemas.rs (538) → move the registered-controller vec (:349-496) into the schemas/ groups.
  • flows/schemas_handlers.rs (568, include!d) → schemas/handlers/{definition,run,builder,draft}.rs.
  • flows/tinyflows/caps/agent.rs (578) → move AgentRoute, route_for_agent_ref, route_custom_entry_lookup, harness_model_default_override (:121-170,:289-306) to flows/agent_route.rs.

ops/builder.rs and ops/builder_gates.rs are split by #69 / #74. RPC names unchanged.

Done when: the rows above are gone from crates/openhuman-core/src/flows/ / crates/openhuman-core/src/cron/ (moved, deleted or reduced to the stated host adapter), with their tests moved alongside. Gates: cargo check (root + crates/openhuman-app), pnpm rust:layout, node scripts/ci/check-feature-forwarding.mjs, pnpm debug rust flows|cron, and the touched crates' tests. RPC namespace strings, model-facing tool names and DomainEvent variant names stay unchanged.

Agent brief

Blocked by #58
Blocked by #60
Parent: tinyhumansai#8055

Activity

  1. added
    refactorStructural code change without a primary behavior change.
    rust-coreCore Rust runtime in src/: CLI, core_server, shared infrastructure.
    priority: p3Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.
    on Oct 16, 2026
  2. senamakel commented on Oct 17, 2026

    @senamakel
    OwnerAuthor

    Implementation ordering update for epic tinyhumansai#8055: #60 is merged. The file splits in this issue do not require the aliases deleted by #58. Under the current bus-only host boundary, the adoption leaves (#68/#69/#70/#71/#72/#74) retire the implementations and callers that #58 originally expected to replace through direct library imports. We will therefore complete these mechanical splits before adoption and reconcile the full #58 cleanup afterward; #58 remains open until every listed row is gone. This preserves both issues' full scope.

    The Schedule v5 cutover in #75 will follow these splits before the new module registry is consumed by other host adopters, preserving cron compatibility during the module upgrade.

  3. senamakel commented on Oct 17, 2026

    @senamakel
    OwnerAuthor

    Merged in #169 (86c1ad8). Mechanical split and existing registrations preserved; builds/static checks passed, broad flow tests 622 passed/1 ignored, and strict Clippy matches the recorded 38-diagnostic baseline. Six parallel profile lease failures are disclosed; all seven lease tests passed serially.

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

    priority: p3Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.refactorStructural code change without a primary behavior change.rust-coreCore Rust runtime in src/: CLI, core_server, shared infrastructure.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions