Skip to content

feat: auto-retry id-like task refs as direct ID lookups#53

Merged
scottlovegrove merged 1 commit intoDoist:mainfrom
mindsocket:fix/parent-list-filtering
Feb 10, 2026
Merged

feat: auto-retry id-like task refs as direct ID lookups#53
scottlovegrove merged 1 commit intoDoist:mainfrom
mindsocket:fix/parent-list-filtering

Conversation

@mindsocket
Copy link
Contributor

@mindsocket mindsocket commented Feb 10, 2026

Fixes #46

Note: Depends on #52 — this PR is built on top of server-side search/filtering improvements. While it could be made independent if is more easily merged after it. See ab3bdc2 for the changes specific to this PR.

Test plan

  • Existing tests pass (675/675)
  • New test: alphanumeric ref auto-retries as direct ID lookup
  • New test: numeric ref auto-retries as direct ID lookup
  • New test: plain text ref does not auto-retry

@mindsocket mindsocket marked this pull request as ready for review February 10, 2026 00:51
@scottlovegrove
Copy link
Collaborator

@mindsocket Would you mind rebasing this branch on main, there are merge conflicts.

When resolveTaskRef fails to match by name and the ref looks like a raw
Todoist ID (per looksLikeRawId), automatically retry with id: prefix
before erroring. Covers both legacy numeric and current alphanumeric IDs.
@mindsocket mindsocket force-pushed the fix/parent-list-filtering branch from ab3bdc2 to f78df78 Compare February 10, 2026 10:22
@mindsocket
Copy link
Contributor Author

@scottlovegrove done!

@scottlovegrove scottlovegrove merged commit bcdb7f3 into Doist:main Feb 10, 2026
3 checks passed
@mindsocket mindsocket deleted the fix/parent-list-filtering branch February 10, 2026 10:41
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.

Unclear param formats for filters can lead to confusion

2 participants

Comments