mirror of
https://github.com/fabro-sh/fabro.git
synced 2026-10-17 04:37:03 +00:00
87dc7de140
The web-install feature was carrying nine pedantic-tier clippy errors from its initial commit. Fix them in place: - \`install.rs\` \`InstallAppState\` switches \`install_token\`, \`storage_dir\`, and \`config_path\` from \`Arc<String>/Arc<PathBuf>\` to \`Arc<str>/Arc<Path>\` so we stop heap-duplicating buffers. - Bring \`Infallible\`, \`axum::middleware\`, \`axum::extract::Request\`, and \`fabro_types::settings::SettingsLayer\` into scope instead of using absolute paths inline. - Replace \`Duration::from_secs(10 * 60)\` with \`Duration::from_mins(10)\`. - \`generate_ephemeral_secret\` never returns \`Err\`; drop the \`Result\`. - \`server/start.rs ensure_storage_server_autostart_allowed\` takes \`Option<&OsStr>\` instead of consuming an \`OsString\` it only reads. - \`server/mod.rs\` storage_dir fallback uses \`map_or_else\` to satisfy \`map_unwrap_or\`. CI now passes \`cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings\` cleanly and the 892-test suite still passes. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| fabro-agent | ||
| fabro-api | ||
| fabro-auth | ||
| fabro-checkpoint | ||
| fabro-cli | ||
| fabro-config | ||
| fabro-core | ||
| fabro-devcontainer | ||
| fabro-github | ||
| fabro-graphviz | ||
| fabro-hooks | ||
| fabro-http | ||
| fabro-install | ||
| fabro-interview | ||
| fabro-llm | ||
| fabro-macros | ||
| fabro-mcp | ||
| fabro-model | ||
| fabro-oauth | ||
| fabro-proc | ||
| fabro-retro | ||
| fabro-sandbox | ||
| fabro-server | ||
| fabro-slack | ||
| fabro-spa | ||
| fabro-store | ||
| fabro-telemetry | ||
| fabro-template | ||
| fabro-test | ||
| fabro-tracker | ||
| fabro-types | ||
| fabro-util | ||
| fabro-validate | ||
| fabro-vault | ||
| fabro-workflow | ||