Skip to content

Storage 8/8: CI gate — no storage-backend deps or *-sqlite/*-mongodb/*-file crates outside tinystoragedrivers #7642

Closed

Description

@senamakel

Part of #7522 (storage: tinystoragedrivers is the only storage owner). The parent issue's Design section is the spec; this issue is one checklist item, done in checklist order. Module/upstream PR first, then the OpenHuman gitlink + host adapter.

Scope (tinyhumansai/openhuman)

  • A script, run as part of pnpm rust:layout, that fails when any crate outside tinystoragedrivers* depends on rusqlite, libsqlite3-sys, mongodb, object_store, sqlx, redb, sled or rocksdb, or defines a *-sqlite / *-mongodb / *-file crate.
  • Allowlist the read-only external-DB readers: tinychannels iMessage, the openhuman-app imessage scanner, and tinymemory legacy-import.
  • Self-test under scripts/__tests__/.

Acceptance

  • Gate script + self-test, wired into rust:layout
  • cargo tree -i rusqlite / -i mongodb / -i object_store show only allowlisted crates

Activity

  1. added
    subtaskSubtask of a larger tracked effort.
    rust-coreCore Rust runtime in src/: CLI, core_server, shared infrastructure.
    on Oct 12, 2026
  2. added
    priority: p3Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.
    on Oct 13, 2026
  3. tinysweeper commented on Oct 13, 2026

    @tinysweeper

    CI gate adding a self-test script to fail builds when crates outside tinystoragedrivers depend on storage backend libraries (rusqlite, mongodb, object_store, etc.) or define -sqlite/-mongodb/*-file crates, with an allowlist for read-only external-DB readers. This is one checklist item in the storage consolidation epic (#7522), an enforcement/build-infrastructure task rather than a user-facing bug or feature. Low urgency: preventive guardrail with no current outage or broken path.

    Labelled priority: p3.

  4. added theissue type on Oct 13, 2026
  5. senamakel commented on Oct 13, 2026

    @senamakel
    MemberAuthor

    #7700: adds scripts/ci/check-storage-owner.mjs + self-test, wired into pnpm rust:layout (#7700)

  6. added a commit that references this issue on Oct 13, 2026
  7. senamakel commented on Oct 13, 2026

    @senamakel
    MemberAuthor

    openhuman#7700 merged (5c2233c): storage:owner check (no storage backend outside tinystoragedrivers*) wired into layout. #7700

  8. senamakel commented on Oct 13, 2026

    @senamakel
    MemberAuthor

    Progress: all acceptance items done and ticked. PR: #7700 (merged). REMAINING: nothing in this issue.

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.rust-coreCore Rust runtime in src/: CLI, core_server, shared infrastructure.subtaskSubtask of a larger tracked effort.

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions