Skip to content

chore(task): remove stale project tasks - #11338

Merged
jdx merged 5 commits into
jdx:mainfrom
risu729:cursor/stale-tasks-tools-75b0
Jul 26, 2026
Merged

jdx merged 5 commits into
jdx:mainfrom
risu729:cursor/stale-tasks-tools-75b0

Conversation

@risu729

@risu729 risu729 commented Jul 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • remove orphaned task-discovery and Windows task fixtures from xtasks/
  • remove the unused manual signal-test fixture and hidden xxx scratch task
  • remove the broken one-off aqua-tester.fish migration helper
  • refresh generated task documentation

Why these became stale

Task-discovery fixtures

Aqua migration helper

Manual scratch tasks

Impact

This removes only repository-local development tasks and fixtures. It does not change CLI behavior. Project tools such as age and sops remain unchanged.

Validation

  • mise run lint-fix
  • git diff --check
  • verified the PR diff contains only task definitions, generated task documentation, and task fixtures

Related

@coderabbitai

coderabbitai Bot commented Jul 26, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Development tooling was reduced by removing obsolete age and sops pins, task definitions, task documentation, fixtures, and helper scripts. A new clean task now runs cargo clean.

Changes

Development task cleanup

Layer / File(s) Summary
Remove obsolete tool pins
mise.toml
Removes the age and sops tool entries.
Update development tasks
tasks.toml, tasks.md, test/fixtures/signal-test.js, xtasks/*
Adds the clean task, removes obsolete task definitions and documentation, and clears the associated fixtures and scripts.

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

A rabbit hops through tasks anew,
cargo clean joins the morning queue.
Old scripts fade without a sound,
While tidy tools leave less around.
Hop, hop—the workspace shines!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: removing stale project tasks and related cleanup.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Jul 26, 2026 •

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes obsolete repository-local task definitions and their associated fixtures and generated documentation.

  • Removes stale task-discovery and Windows fixture scripts from xtasks/.
  • Removes the unused signal-test and hidden xxx tasks.
  • Removes the obsolete Aqua migration helper.
  • Refreshes tasks.md to match the remaining task definitions.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
tasks.toml Removes the unused signal-test and hidden xxx project tasks while retaining the existing release task.
tasks.md Removes generated documentation for the deleted Aqua, file-task, and signal-test tasks.
xtasks/aqua-tester.fish Deletes an obsolete Aqua migration helper whose external input no longer exists.
xtasks/filetask Deletes an orphaned task-discovery fixture and its associated platform-specific fixtures.
test/fixtures/signal-test.js Deletes the fixture used only by the removed manual signal-test task.

Reviews (4): Last reviewed commit: "chore(task): narrow stale task cleanup" | Re-trigger Greptile

Comment thread docs/contributing.md Resolved
cursoragent and others added 4 commits July 27, 2026 06:52
Drop leftover fixtures and unused tooling similar to the recent
update-descriptions / fd / ripgrep cleanups: unused age/sops project
tools, the dead cargo-release release task, and xtask test fixtures
that are no longer referenced.

Co-authored-by: Taku Kodama <risu729@users.noreply.github.com>
Avoid overlapping that PR's mise.toml/lock tool cleanup; keep the
dead [release] task and nix leftover fixes here.

Co-authored-by: Taku Kodama <risu729@users.noreply.github.com>
@risu729 risu729 changed the title chore: remove stale project tasks and tools chore(task): remove stale project tasks Jul 26, 2026
@risu729
risu729 marked this pull request as ready for review July 26, 2026 21:03
@jdx
jdx merged commit 5530d2b into jdx:main Jul 26, 2026
34 checks passed
@risu729
risu729 deleted the cursor/stale-tasks-tools-75b0 branch July 26, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants