Skip to content

[tiny-modules] tinyskills: move the per-run skill log format into tinyskills #7570

Closed

Description

@senamakel

Parent: #7531

Context

hosting.enabled (config/schema/hosting.rs:22, checked at hosting/mod.rs:50) exists, but hosting has no DomainSet group and no ToolGroup, and its registry record is compiled and trusted regardless.

What moves

  • [hosting] enabled folds into [components] hosting (migration)
  • Add the hosting tools to a ToolGroup and a DomainSet/ComponentSet entry so hosts and config can narrow them
  • modules::ensure_loaded("tinyhosts") refuses when off

What stays in OpenHuman

Wiring only; credential gating in tools/ops.rs stays.

Upstream target

None (host-only). Uses the E1 framework.

Host adapter after the move

Off ⇒ no module load, no hosting tools, modules.list reports disabled.

Acceptance criteria

  • Host-only change: no upstream PR or module release needed for the toggle itself (the module keeps its released pin)
  • One-shot config migration from the old key(s), with a test
  • Component off: module never downloaded or loaded, no tools, no background services, RPC namespace answers COMPONENT_DISABLED, modules.list/about_app show it disabled
  • Component on: behaviour unchanged
  • pnpm rust:layout, cargo check --manifest-path Cargo.toml, cargo check --manifest-path crates/openhuman-app/Cargo.toml, affected pnpm debug rust <filter> green
  • Tests: component off ⇒ no hosting tools and no module load; old hosting.enabled=false migrates

Depends on

#7525 (E1 [components] framework, leaf E1.1)

Activity

  1. added
    subtaskSubtask of a larger tracked effort.
    refactorStructural code change without a primary behavior change.
    rust-coreCore Rust runtime in src/: CLI, core_server, shared infrastructure.
    skillsQuickJS skills runtime, manifests, sync, and skill tooling in src/openhuman/skills/.
    on Oct 12, 2026
  2. senamakel commented on Oct 13, 2026

    @senamakel
    MemberAuthor

    Module side merged: tinyhumansai/tinyskills#33 (merge 5f0361e). Released https://github.com/tinyhumansai/tinyskills/releases/tag/v0.4.0 (tag 27cf6d0, no .crate asset; cargo package cannot resolve the tinytools git dep, fix in tinyskills#34). Note: search::project takes the sanitiser as a parameter (host passes tinytools_std sanitize_for_llm) to keep reqwest out of the crate. Host adapter swap and gitlink repin to v0.4.0 remain.

  3. senamakel commented on Oct 13, 2026

    @senamakel
    MemberAuthor

    Done in #7681 (merged, b80e23c); module PR tinyhumansai/tinyskills#33, release v0.4.0.

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

    refactorStructural code change without a primary behavior change.rust-coreCore Rust runtime in src/: CLI, core_server, shared infrastructure.skillsQuickJS skills runtime, manifests, sync, and skill tooling in src/openhuman/skills/.subtaskSubtask of a larger tracked effort.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions