drew/skate
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Set default path.

Browse source
This commit is contained in:
Drew 2026-02-25 22:55:34 -08:00
parent 7efc6705d3
commit 023156ef4e
2 changed files with 19 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

18
TODO.md
View file

@ -1,10 +1,16 @@
# Cleanups # UI Sync
- Parallelize tool-use execution in `run_turn` -- requires refactoring `Orchestrator` to use `&self` + interior mutability (`Arc<Mutex<...>>` around `event_tx`, `action_rx`, `history`) so multiple futures can borrow self simultaneously via `futures::future::join_all`.
- Move keyboard/event reads in the TUI to a separate thread or async/io loop
- Keep UI and orchestrator in sync (i.e. messages display out of order if you queue up many.) - Keep UI and orchestrator in sync (i.e. messages display out of order if you queue up many.)
- `:clear` clears TUI state immediately but sends `ClearHistory` to orchestrator async. A mid-stream response can ghost back in after clear. Need synchronization (e.g. clear on `TurnComplete`, or have orchestrator ack the clear).
# Scroll
- `update_scroll` auto-follows in Insert mode, yanking viewport to bottom on mode switch. Only auto-follow when new content arrives (in `drain_ui_events`), not every frame. - `update_scroll` auto-follows in Insert mode, yanking viewport to bottom on mode switch. Only auto-follow when new content arrives (in `drain_ui_events`), not every frame.
- `G` sets scroll to `u16::MAX` and relies on `update_scroll` clamping. Compute actual max_scroll inline, or document the contract that `update_scroll` must always run before render. - `G` sets scroll to `u16::MAX` and relies on `update_scroll` clamping. Compute actual max_scroll inline, or document the contract that `update_scroll` must always run before render.
- `:clear` clears TUI state immediately but sends `ClearHistory` to orchestrator async. A mid-stream response can ghost back in after clear. Need synchronization (e.g. clear on `TurnComplete`, or have orchestrator ack the clear).
- Command overlay width: `(out.width / 2).max(80)` makes overlay full-bleed on 80-col terminals. Consider `.max(40)` or a different minimum. # Cleanups
- Move keyboard/event reads in the TUI to a separate thread or async/io loop
- When a permission is denied, output a useful tools message to the agent (i.e. searching the internet is not enabled).
- Parallelize tool-use execution in `run_turn` -- requires refactoring `Orchestrator` to use `&self` + interior mutability (`Arc<Mutex<...>>` around `event_tx`, `action_rx`, `history`) so multiple futures can borrow self simultaneously via `futures::future::join_all`.

10
src/main.rs
View file

@ -12,7 +12,7 @@ use anyhow::Context;
/// Run skate against a project directory. /// Run skate against a project directory.
/// ///
/// ```text /// ```text
/// Usage: skate --project-dir <path> [--yolo] /// Usage: skate [--project-dir <path>] [--yolo]
/// ``` /// ```
/// ///
/// `ANTHROPIC_API_KEY` must be set in the environment. /// `ANTHROPIC_API_KEY` must be set in the environment.
@ -35,7 +35,8 @@ struct Cli {
/// Parse `argv` into [`Cli`]. /// Parse `argv` into [`Cli`].
/// ///
/// Accepts `--project-dir <path>` (required) and `--yolo` (optional). /// Accepts `--project-dir <path>` (optional, defaults to current directory)
/// and `--yolo` (optional).
fn parse_cli() -> anyhow::Result<Cli> { fn parse_cli() -> anyhow::Result<Cli> {
let mut project_dir: Option<PathBuf> = None; let mut project_dir: Option<PathBuf> = None;
let mut yolo = false; let mut yolo = false;
@ -56,6 +57,9 @@ fn parse_cli() -> anyhow::Result<Cli> {
} }
} }
let project_dir = project_dir.context("Usage: skate --project-dir <path> [--yolo]")?; let project_dir = match project_dir {
Some(p) => p,
None => std::env::current_dir().context("failed to determine current directory")?,
};
Ok(Cli { project_dir, yolo }) Ok(Cli { project_dir, yolo })
} }