Repository navigation
chore(tinytools): remove tinytools-std - #99
Merged
senamakel merged 8 commits intoOct 24, 2026
Merged
Conversation
The tinytools-std crate and all of its tool implementations have been deleted, leaving the workspace with the tinytools contract definitions only. The README now states that concrete tools belong in their owning host or module rather than in this repository. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Update the workspace guidance, the web_fetch status module docs, and the host-adoption map to stop pointing at `tinytools-std`, which no longer exists. The shared tool definitions and call protocols now live in this workspace while concrete tools and host-independent building blocks belong in their owning host or module repository. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Regenerate Cargo.lock after removing dependencies that are no longer referenced by the workspace, dropping transitive crates such as atomic-waker, base64, and the crypto stack. This shrinks the lockfile and keeps it in sync with the current manifest. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Tiny Sweeper reviewReviewed head: Verdict: Ready for maintainer review · Priority: none Diagnostics and supporting detailsFull summaryPull request purpose (title): chore(tinytools): remove tinytools-std. The pull request removes the `tinytools-std` crate so TinyTools ships only shared tool definitions and call protocols, with concrete tools owned by hosts and module repositories. It deletes the crate tree plus its workspace and lockfile entries, retargets the workspace documentation, drops the `web_fetch` status alias, and marks four plan/spec documents historical or superseded. Unreviewed paths
Review snapshot
Completeness: Complete What changedThe `tinytools-std` crate and its `Cargo.lock` entries are removed, so the host-independent building blocks it provided no longer ship from this workspace: the filesystem tools behind an `FsGate`, the network tools behind a `NetGate`, the URL/SSRF guard, `file_state` tracking, PATH probing, and timeout/process-group management. `AGENTS.md` now states the workspace holds shared definitions and call protocols only, and `README.md` scopes the crate to definitions with a "No tool implementations" note. The rustdoc in `crates/tinytools/src/status/web_fetch.rs` is reworded to drop the `tinytools-std` reference with parser behavior unchanged, and `docs/host-adoption/72-host-adoption-map.md` no longer documents the removed `status_of` alias. Five plan/spec documents gain historical or superseded notes pointing to `tinyagents-harness-tools`. The description says downstream pins must be coordinated before merge, and the tests lane noted it could not verify whether `.github/workflows/ci.yml` still references the removed crate because that file is not in the diff. Features
TestsNo supported feature-to-test mapping was produced. Test execution is not inferred. FindingsNo active actionable findings. Before mergeNone. Agent review detailscritique
security
tests
commits
description
Evidence and run details
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Warning Review limit reached
This review includes 116 billable files and costs up to $29.00. View limit details
Comment |
![tinysweeper[bot]](https://avatars.githubusercontent.com/in/4526585?s=60&v=4){kind=small}
Co-authored-by: Medulla <medulla@tinyhumans.ai>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 34956944c0
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Add historical and superseded notes to the collapsed-tools, web-fetch, and fsgate documents, recording that the tinytools-std work was removed in tinytools#99 and its helper implementations are now owned by tinyagents-harness-tools. The status lines were adjusted so these documents no longer claim to describe APIs currently implemented by tinytools. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TinyTools now keeps shared definitions and call protocols; concrete tools live in their owning hosts and modules. This removes the unused
tinytools-stdcrate, its workspace and lockfile entries, and updates the documentation andweb_fetchstatus alias.Closes senamakel#5.
This is part of tinyhumansai/openhuman#8088. The downstream TinyAgents, TinyMCP, and OpenHuman integrations are tracked in the epic; coordinate their final pins before merging this removal.
Validation on the source PR branch:
cargo fmt --all -- --checkcargo check --workspace --all-targetscargo clippy --workspace --all-targets -- -D warningscargo test --workspacerg tinytools_stdin TinyTools: no hits