Skip to content

fix(scheduled): use project path identity for task bindings - #820

Merged
vastsa merged 1 commit into
vastsa:mainfrom
hui455:fix/scheduled-paths-final
Sep 21, 2026
Merged

vastsa merged 1 commit into
vastsa:mainfrom
hui455:fix/scheduled-paths-final

Conversation

@hui455

@hui455 hui455 commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

On Windows, a task created in the Scheduled page can be missing from an AI conversation in the same project. The page still shows the task, but the conversation cannot list, update or delete it because the saved paths use different spellings.

Use PI-Desktop's existing project path normalization when reading and saving task bindings. Same-project conversations can manage the task after restart; other projects remain isolated. Legacy unbound tasks and tasks explicitly created without a project keep their existing behavior.

Before and after — actual Windows desktop (1200 × 800, isolated profiles). A deterministic local model fixture calls the real tools and summarizes their results; no live model is used.

Before:

Before

After:

After

Regression tests reproduce the visibility failure and cover Windows path aliases, cross-project rejection, all four cadences and reopening SQLite.

Validation details
  • Scheduled Rust tests: 24 passed. Real Host E2E and the workspace/hourly compatibility suites passed.
  • Full Host suite on the refreshed base: 573 passed, 4 failed outside scheduled tasks (MCP/Skills path assertions and session attachment-fork tests).
  • Formatting and diff checks passed.
  • Windows, Node 24.2, Rust debug build. Native macOS/Linux runs were not performed. Screenshots use main 0111e306c and the corresponding fix; focused checks were rerun after refreshing the candidates to 46d4ee4ee.

Use the project storage identity when reading and writing scheduled
bindings so Windows path aliases do not hide tasks from their sessions.
Preserve legacy unbound and explicitly projectless tasks.
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.

2 participants