fix(tinyflows): restore compat and preflight module declarations dropped in merge

[senamakel]

Summary

• Merge commit fb946d6 ("Merge remote-tracking branch 'refs/remotes/upstream/main' into pr/78") silently dropped the pub mod compat; and #[cfg(any(test, feature = "mock"))] pub mod preflight; declarations from crates/tinyflows/src/lib.rs, as collateral damage of a botched unrelated-history merge (the merge's second parent, 16aad9a, came in via a conflict resolution that regenerated lib.rs from a base treated as empty — note the 0000000 base blob in git show fb946d6 -- crates/tinyflows/src/lib.rs).
• The commit immediately before the merge (7601283) still declared both modules. The implementation files (compat.rs, compat_tests.rs, preflight.rs, preflight_tests.rs) were never deleted and still compile — only the pub mod declarations vanished from lib.rs.
• As a result, tinyflows v0.8.1 shipped without this public API, even though the crate still ships the implementation. This broke downstream consumers that import tinyflows::compat::… and tinyflows::preflight::… (e.g. openhuman).
• This PR restores the two dropped declarations verbatim (doc comments and cfg gates matching the pre-merge state exactly), back in their original position in lib.rs.

Also investigated: browser and companion

The same merge also dropped pub mod browser and pub mod companion (both #[cfg(feature = "chrome-extension")]). Unlike compat/preflight, these were not accidental collateral damage — their source files (browser.rs, companion.rs) no longer exist anywhere in history. They were deliberately removed by PR #79 ("remove-chrome-extension", commit 4d6b874 "refactor: remove Chrome workflow companion"), which is the merge's other parent (16aad9a). This PR intentionally leaves browser/companion untouched.

Test plan

• git show fb946d6 -- crates/tinyflows/src/lib.rs — confirmed the diff drops both declarations against a 0000000 base
• git show 7601283:crates/tinyflows/src/lib.rs | grep -n "pub mod" — confirmed both declarations existed pre-merge
• Confirmed compat.rs, compat_tests.rs, preflight.rs, preflight_tests.rs still exist on upstream/main and are unreferenced by any other mod declaration
• Confirmed browser.rs/companion.rs do not exist anywhere in history (removed by PR Remove Chrome workflow companion #79) — left alone
• cargo check -p tinyflows --features mock
• cargo check --workspace --all-features
• cargo fmt --all -- --check
• cargo clippy --all-targets --all-features -- -D warnings

Summary by CodeRabbit

• New Features
  • Added workflow compatibility checks to identify unsupported fan-in configurations before execution.
  • Added preflight validation for outbound argument resolution before workflows are saved.
• Bug Fixes
  • Improved safeguards against invalid workflow configurations reaching execution or persistence.

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-08-30T20:46:11.347057Z	ae41aca	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[tinysweeper]

tinysweeper found nothing blocking. Approving.

$0.0000 · 0 in / 0 out

[tinysweeper]

How this change flows

0 changed behaviours across 1 relationship. 2 surrounding behaviours are shown (60 graph nodes walked). 4 further behaviours left out to keep the diagram readable.

flowchart LR
  n0["node_of"]:::impacted
  n1["model"]:::impacted
  n0 -->|uses| n1
  classDef changed fill:#0d4429,stroke:#238636,color:#e6edf3
  classDef impacted fill:#161b22,stroke:#6e7681,color:#c9d1d9
  classDef flagged fill:#5a1e02,stroke:#d93f0b,color:#ffffff
  classDef blocking fill:#67060c,stroke:#f85149,color:#ffffff

Loading

_{Green: changed behaviour. Grey: surrounding behaviour. Arrows name the call, use, implementation, or test relationship. Orange: has findings. Red: has a finding that blocks the merge.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d2ede0e8-c458-48b3-ac2d-33dbe9a96a9d

📥 Commits

Reviewing files that changed from the base of the PR and between 2c2d02a and ae41aca.

📒 Files selected for processing (1)

• crates/tinyflows/src/lib.rs

---

📝 Walkthrough

Walkthrough

The crate now publicly exposes compatibility validation and feature-gated preflight validation modules. The compatibility module checks unsafe fan-in topologies before execution. The preflight module validates outbound workflow arguments before persistence when tests or the mock feature are enabled.

Changes

Workflow validation

Layer / File(s)	Summary
Compatibility validation export crates/tinyflows/src/lib.rs	The crate exposes compat for rejecting unsafe fan-in topologies before execution.
Preflight validation export crates/tinyflows/src/lib.rs	The crate exposes preflight under test or mock feature gates for mock-based outbound argument validation before saving.

Estimated code review effort: 2 (Simple) | ~5 minutes

Poem

A rabbit checks the flow

Fan-in paths must safely go
Mocked args wait before they save
Feature gates keep checks in place
Tiny workflows run brave

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

Warning

Your free Security trial is over. An organization admin can activate Security or dismiss this notice.

---

_{Comment @coderabbitai help to get the list of available commands.}