Skip to content

fix(channels): verify real WeChat task endpoint#61

Merged
acedatacloud-dev merged 1 commit into
mainfrom
fix/doctor-windows-install
Jul 13, 2026
Merged

fix(channels): verify real WeChat task endpoint#61
acedatacloud-dev merged 1 commit into
mainfrom
fix/doctor-windows-install

Conversation

@acedatacloud-dev

Copy link
Copy Markdown
Member

Root causes

  • get_task_status called /api/messages/tasks/{id}, but Wisdom exposes /api/tasks/{id}. The missing route returned 404 before auth, so doctor falsely accepted wrong tokens.
  • Local Windows install was a stale editable 0.1.0 pointing at a deleted worktree; pip said satisfied while the module was absent.
  • Module version was hard-coded 0.1.0 even in CalVer wheels.

Fix

  • Use /api/tasks/{id} throughout client/docs/tests.
  • Runtime version comes from wheel metadata.
  • Add Windows stale-editable recovery + PowerShell/CMD token syntax.

Validation

  • ruff clean; 29 focused tests pass; wheel builds.
  • Real gateway: correct token doctor exit0; wrong token 401/exit1.
  • Real channels start: WebSocket connected; token redacted.

Adversarial review

VERDICT CLEAN. Endpoint/auth/version/install/secret handling reviewed.

@acedatacloud-dev acedatacloud-dev merged commit ef0ed79 into main Jul 13, 2026
3 checks passed
@acedatacloud-dev acedatacloud-dev deleted the fix/doctor-windows-install branch July 13, 2026 01:34
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.

1 participant