Skip to content

修复主站与编辑器交互竞态 - #21

Merged
4evour merged 1 commit into
mainfrom
codex/pr-frontend-race-fixes
Aug 22, 2026
Merged

修复主站与编辑器交互竞态#21
4evour merged 1 commit into
mainfrom
codex/pr-frontend-race-fixes

Conversation

@4evour

@4evour 4evour commented Aug 21, 2026

Copy link
Copy Markdown
Owner

迁移内容

  • 将动态内联点击处理改为 CSP 兼容的事件委托
  • 为路线和酒店请求增加取消机制,避免旧响应覆盖新状态
  • 向路线查询传递城市上下文,并实际挂载路线钩子
  • 缩短总天数时同步裁剪旧日期
  • 登录前后侧栏仅保留已上线入口并保持一致
  • 更新游客结构化规划 UI 冒烟脚本,并重新构建编辑器发布包

验证

  • node tests/test_reviewed_frontend_contracts.js
  • node tests/test_tour_ai_layout_markup.js
  • npm run editor:test(17 项通过)
  • npm run editor:build
  • git diff --check

依赖关系

这是第 4 个堆叠 PR,基于 POI 一致性 PR #20。请按 #18#19#20 → 本 PR 的顺序合并;前序合并后将本 PR 的目标分支改回 main。

已知事项

npm ci 报告 2 个高危依赖告警,本 PR 未自动升级依赖,避免把依赖升级混入竞态修复。

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@4evour, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 28 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 42e06ab5-ca35-482a-8b40-bec52d59a3db

📥 Commits

Reviewing files that changed from the base of the PR and between 0867da7 and 7640406.

📒 Files selected for processing (13)
  • CHANGELOG.md
  • scripts/verify_ui.js
  • tests/test_reviewed_frontend_contracts.js
  • tests/test_tour_ai_layout_markup.js
  • web/app.js
  • web/editor-dist/assets/index-B1QacJb9.js
  • web/editor-dist/index.html
  • web/editor/src/NewEditorApp.tsx
  • web/editor/src/components/Wizard/HotelsStep.tsx
  • web/editor/src/hooks/useRoute.ts
  • web/editor/src/stores/itineraryStore.test.ts
  • web/editor/src/stores/itineraryStore.ts
  • web/index.html

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.

@4evour
4evour force-pushed the codex/pr-poi-transactional-updates branch from 4ca93d4 to 0bc05f9 Compare August 22, 2026 10:12
@4evour
4evour force-pushed the codex/pr-frontend-race-fixes branch from bd1f901 to 7640406 Compare August 22, 2026 10:12
@4evour
4evour changed the base branch from codex/pr-poi-transactional-updates to main August 22, 2026 10:21
@4evour
4evour marked this pull request as ready for review August 22, 2026 10:21
@4evour
4evour merged commit d6b83c0 into main Aug 22, 2026
1 check passed
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