Skip to content

test: minimal browser acceptance for #214 #218 #219 - #240

Merged
lyyzka merged 1 commit into
mainfrom
codex/214-218-219-minimal-browser
Oct 20, 2026
Merged

lyyzka merged 1 commit into
mainfrom
codex/214-218-219-minimal-browser

Conversation

@lyyzka

@lyyzka lyyzka commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Scope

Independent acceptance-only follow-up for #214, #218, and #219, based on main@754cd873. No product component, dependency, migration, or release changes.

Verification and limits

  • Public e2e@0.17.0 / @e2e-dev/web@0.12.0 selected-file TypeScript check passed; official e2e list --reporter json selected 6 tests with disposition run, 0 skipped. node --check e2e/server/forum.mjs and git diff --check passed.
  • Product browser results: 0/6 run here. This checkout's product dependencies are incomplete after the earlier private @lyyzka/lingxios@3.5.6 E401 Unauthorized. I did not retry installation or copy credentials. Previous live official-reference navigation failed ERR_CERT_AUTHORITY_INVALID; I did not bypass TLS. Native sandbox success is unverified here.
  • The smallest replay commands and exact fixture/provider boundaries are in the handoff README. The official live comparison and product browser outcome still require a normally provisioned environment. fix(visualize): 放宽触发、复用开源实现渐进加载,优化进入速度并将工具栏移至右上角 #217 quality/permission work remains separate.

Draft for integration review only. Do not merge, deploy, publish, or close issues from this PR.

lyyzka commented Oct 20, 2026

Copy link
Copy Markdown
Contributor Author

Root已独立审查exact7769b041的9个文件,并在现有正式依赖环境完成完整 tsc -p e2e/tsconfig.json(exit0)。官方e2e分别选中1+2+2+1=6例,全部run disposition;list不是pass。

实际浏览器执行仍0/6、通过0。2026-10-20T16:10:12UTC使用既有Chromium1243、新临时profile、headless/native reduced-motion、loopback CDP53129启动一次,exit134 / No usable sandbox。未进入测试断言;已停止此路径,没有关闭sandbox、改userns/AppArmor、忽略TLS或使用无沙箱替代。官方live对照的既有TLS限制没有新恢复证据,未重试官网。

精确启动stderr、六例名单、类型检查与后续命令。下一步最小依赖是可用且保持沙箱的测试浏览器/native media+CDP;218同样可给现有root config传E2E_CDP_URL。按本PR README仅跑六例;219forum用既有隔离本地PG/Redis,不需生产登录或新身份。没有把环境阻塞报告为产品功能缺陷或基本验收通过。

@lyyzka
lyyzka marked this pull request as ready for review October 20, 2026 16:28
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-20T16:31:37.642503Z 7769b04 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@lyyzka
lyyzka merged commit 329366d into main Oct 20, 2026
7 checks 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.

2 participants