diff options
| author | Paul Buetow <paul@buetow.org> | 2026-03-23 22:41:39 +0200 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-03-23 22:41:39 +0200 |
| commit | 5beb39a3338e83c9a5906d2e5f7acb3bf795811d (patch) | |
| tree | 0777ad808aa9e8dd5ae9b3a6c149a2b8eac579d7 /internal/askcli/taskexec_test.go | |
| parent | b18ee2f6b6e0cb7164a7bbbc8efbb4c75cffade6 (diff) | |
fix: prevent ask add from swallowing tags into descriptions, add rc.confirmation=offv0.25.11
parseAddArgs now rejects args that start with '+'/'-' but contain spaces as
modifiers — those are description text, not tags (tags cannot have spaces).
This prevents agents from quoting tag+description together and having the tag
silently land in the task description with no tag applied.
Also removed the fallthrough that duplicated all-modifier args as description.
Added rc.confirmation=off to every taskwarrior invocation so that write
operations (done, delete, start, etc.) succeed non-interactively when stdin
is unavailable (as is always the case when called from an agent).
Bump version to v0.25.11.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/askcli/taskexec_test.go')
| -rw-r--r-- | internal/askcli/taskexec_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/internal/askcli/taskexec_test.go b/internal/askcli/taskexec_test.go index 4450a28..5cbb7f1 100644 --- a/internal/askcli/taskexec_test.go +++ b/internal/askcli/taskexec_test.go @@ -17,7 +17,7 @@ func TestExecutorTaskArgs(t *testing.T) { if err != nil { t.Fatalf("taskArgs returned error: %v", err) } - want := []string{"project:hexai", "+agent", "list", "limit:1"} + want := []string{"rc.confirmation=off", "project:hexai", "+agent", "list", "limit:1"} if !reflect.DeepEqual(args, want) { t.Fatalf("task args = %v, want %v", args, want) } @@ -47,7 +47,7 @@ func TestExecutorRun_InjectsProjectFilterAndAgentTag(t *testing.T) { if gotName != "/usr/bin/task" { t.Fatalf("task binary = %q, want /usr/bin/task", gotName) } - wantArgs := []string{"project:hexai", "+agent", "list", "limit:1"} + wantArgs := []string{"rc.confirmation=off", "project:hexai", "+agent", "list", "limit:1"} if !reflect.DeepEqual(gotArgs, wantArgs) { t.Fatalf("task args = %v, want %v", gotArgs, wantArgs) } |