refactor: remove legacy Celery naming references#1
Merged
Conversation
Celery+Redis was replaced with an in-process ThreadPoolExecutor long ago, but the old naming persisted in file names, imports, types, and docs. - Rename backend/src/celery_app.py → backend/src/task_executor.py - Update all imports across 6 router/main modules - Rename celery_task_id → task_id in frontend types and E2E fixtures - Update mock error message in git-sync E2E test - Update CLAUDE.md file path references The DB column name (celery_task_id) is kept as-is to avoid a migration. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
raffelino
added a commit
that referenced
this pull request
May 7, 2026
…s, dashboard screenshot
Last sweep before release:
1. **Recorder docs — selector verification & Shadow DOM**
New `recorder-selector-verification` topic in all 4 langs sitting
between `recorder-anatomy` and `recorder-extension`. Covers:
- Visibility-aware uniqueness ranking ({ total, visible,
actionable }) with the gold / verified / hidden / multi-match
tiers + their score penalties.
- Parent-context CSS disambiguation (the
`#checkout-form button.submit-btn` rewrite that prevents the
#1 Playwright strict-mode failure source at replay).
- Shadow DOM aware capture (`composedPath()[0]` retargeting,
ancestor walk crossing shadow boundaries via the host) and
the `host >> inner` chained Playwright locator emitted by the
synthesis layer when `in_shadow_dom` is set.
- Closed-shadow-root caveat: closed roots are opaque to userspace
JS, so closed-root elements fall back to the host selector.
2. **CLAUDE.md — four new critical-pattern gotchas**
Added to the "Critical patterns & gotchas" list:
- Setting-meta side-note inputs MUST use a draft buffer (mirrors
the cloneStep contract for step args; if a future panel
v-models straight into `props.form` the deep watcher tears
`selectedNode` down on every keystroke).
- Setting-meta stacking pitch (`META_PITCH = 96`) + CSS
`max-height: 76px` + line-clamp 2 are tuned together; bumping
pitch lower without tightening clamp lets [Documentation]
overflow into the [Tags] node below.
- Capture script MUST use `realTarget(ev) = composedPath()[0]`
so events inside an open shadow root capture the real target,
not the host. Any new event listener must route through it or
shadow-DOM clicks fire on the wrong element.
- Selector synthesis MUST emit a parent-context CSS variant
when an ancestor has a stable id / data-testid; the verifier's
`nth=0` rewrite is a last-resort fallback, not a substitute.
3. **README dashboard screenshot regenerated**
`docs/screenshots/dashboard.png` was from 24 Feb. and showed the
pre-0.9.0 KPI / recent-runs / repo-grid layout. Replaced with a
fresh 1280x800 capture of the card-grid + Tip-of-the-Day
landing page that 0.9.0 ships.
Topic counts after this commit: EN/DE 91, FR/ES 92.
vue-tsc clean; 491 frontend specs green.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
backend/src/celery_app.py→backend/src/task_executor.pyand updates the module docstringmain.pycelery_task_id→task_idin frontend types and E2E test fixturesgit-sync.spec.ts(Celery broker unreachable→Task executor unavailable)CLAUDE.mdfile path and section referencesThe DB column name (
celery_task_id) is intentionally kept as-is to avoid requiring a migration.Test plan
rg -i celery backend/src/ frontend/src/ e2e/tests/returns zero matches (except DB column name in models.py)make test-backend)execution-analysis,git-sync)🤖 Generated with Claude Code