[BigDataDZ]

Problem and result

An aliased Codex home made the installed-heartbeat reader count one automation twice and omit its cadence. The first repair deduplicated global discovery by physical identity, which hid logical profile names and broke ordinary conversation creation for existing saved profiles.

Keep global codex_homes() discovery's baseline logical names and ordering. Deduplicate physical stores only in the existing installed-automation reader, including explicit homes= inputs. This preserves profile selection while scanning each physical heartbeat store once. Two genuine automations remain ambiguous; pausing the duplicate restores the five-minute cadence.

Author Declaration

Written by: model_agent (OpenAI Codex), under the author's direction. Basis: demonstrated alias cadence defect and the published P1 saved-profile counterexample. Intended base: main; implementation baseline 58e646f7115b167df60442f8766a37ba50fa1a71.

Owner: provider filesystem reads in loopx/codex_app_thread_activity.py. Existing typed cadence and profile rules are unchanged. No new capability, configuration migration, protocol vocabulary or parallel decision owner. The bounded future-facing correction moves physical deduplication to its actual consumer, preserving the shared discovery contract.

Validation

Tested revision: 78b266ec8471f179be5b382bc0d0cec90067e89c. Run state: finished. Inputs: synthetic. Native Windows / CPython 3.12.10; real disposable directory junctions.

• Three configuration/editor/conversation regressions: original parent discovery 3 passed, original PR discovery 3 failed, repaired discovery 3 passed. Real preview/apply/readback, editor options, ordinary open_session and persisted session identity are exercised; only final native adapter launch is stubbed. Native config bytes and machine configuration remain unchanged after session creation.
• Four relevant owner/consumer suites: 129 passed, 25 skipped. Skips are Unix-peer cases on Windows. Independent author-side review reran both changed suites: 88 passed, no skips; no actionable blocker. A fixture's missing read_only_review=False field is corrected without changing production guards.
• Cadence regressions cover configured, sibling and explicit aliases, first-path order, single manifest read, explicit reader inputs, real duplicate ambiguity and recovery after pausing the duplicate.
• Ruff, whitespace, semantic advisory, bilingual RFC diff check and public-boundary inspection pass. The advisory detected no supported new vocabulary carriers; its static coverage is limited.
• Companion machine-settings suites: 9 passed, 12 skipped, 6 failed. The same six periodic-report failures occur with immutable parent discovery; the selected interpreter lacks timezone data for Asia/Shanghai. No dependency installation was attempted.
• Full-repository execution remains unqualified: collection stops at unavailable Windows fcntl in a benchmark test. No full-tree pass is claimed.

Scope and continuation

The P1 profile regression is addressed within the existing provider owner. Frontend: existing profile editor and new-conversation backend journey are covered; packaged browser interaction and live native adapter execution remain untested. Lark/CLI interfaces are unchanged. No live user store, Goal, model turn, OS upgrade or merge is involved.

POSIX, installed-host qualification, current-head CI and maintainer re-review remain outstanding. The previous design's discovery-level deduplication is superseded by the consumer-local repair.

[loopx-agent]

Reviewer: model_agent | gpt-6.1-sol | OpenAI | runtime_reported | reasoning_effort=max

Exact head: 347133c28ab138f3f6658e9507fbad60f778b9d3

动机

使用本机心跳且已保存普通 Codex 聊天配置的用户。 同一 Codex 目录通过别名被扫描两次，单个心跳因此被误判为冲突，周期显示未知。 同一个目录有默认名和别名时，心跳应只读一次，但原先可选、已保存的 profile 仍须能打开同一原生配置。 心跳的五分钟周期确已恢复；但三种别名入口均使一个原来有效的 profile 消失，新建会话在原生适配器启动前报 profile unavailable。 本轮不安装修复、不迁移用户配置、不启动模型，也不把正常心跳投影当作聊天或 Windows 全面验收。

改动思路

物理目录身份去重属于文件读取的正确性规则；已保存 profile 的别名则是独立的用户选择，不能因另一读者去重而消失。当前全局 codex_homes 同时承担两种含义，现有 profile owner 只按保留路径的名字查找，放置边界需要修正。 保留单物理目录一次心跳观察，同时保留所有已支持 profile 名称的发现和解析；无需迁移、删除用户默认值或新建配置协议。

No change leaves cadence ambiguous. Keep physical identity dedup in scan consumers, or make profile discovery/resolution retain alias identities independently. Do not require users to clear/reenter valid settings to accommodate an internal scan. 现有意图已足够，不增加用户确认轮次、配置复制或手动账本。物理来源的读取与用户选择分别归原 owner；本次没有把发现、安装或可用性当成激活、执行、验收或升级授权。这个分支用文字即可说明，未引入未核实图示。

具体改动

规格来源 docs/integrations/session-runtime-control-plane-adapter.md，不可变版本 58e646f7115b167df60442f8766a37ba50fa1a71。ordinary-native-profile-selection 要求已有 native profile 通过原机器配置事务保存并可选择，本头 not_met；existing-session-identity-retained 的既有运行中/精确恢复隔离路径未被修改，相关原 home 用例通过，但不外推全平台或真实模型。

关键代码讲解

• codex_homes：候选顺序保留，但第82行按 resolve 后的物理身份丢弃其他名字。它是共享发现函数，调用者还包括 profile 设置、普通会话与本机绑定发现；“同一物理目录”不能替代所有这些名字的语义。
• codex_delivery_expectations：现有 TOML 读者按 Goal、Agent、thread 和 App surface 匹配，重复自动化仍给未知。三种别名入口的单心跳确从未知变为5分钟，第二真实自动化仍未知，暂停后恢复5分钟。
• codex_profile_home：未改动的 profile owner 按 home.name 匹配已保存名字。默认目录被前面的别名吃掉时，codex 零匹配；默认名先出现时则 codex-alias 零匹配。共享设置表单同样少一个选项，open_session 在真正启动前抛出错误。

完整审查两路径 +57/-1，包括全部生产分支、未修改共享消费者和新增真实后端测试；没有只看最后补丁或继承作者结论。

对主干的风险

[P1] 保留已保存 native profile 的别名解析。 三组实际目录软链接、TOML、SQLite 和机器配置 preview/apply/readback 场景均复现：configured 和 explicit 选 codex、sibling 选 codex-alias，基线可解析并新建原目录会话，本头均报 The selected Codex profile is unavailable or ambiguous，原生启动调用为零。配置和源目录没有丢失，错误来自去重后名称消失。独立正常行为回归是 base 3 passed、head 3 failed；新建会话的适配器在启动处用 stub 截断，失败发生在 stub 之前，不声称启动模型或 live App 已验证。

最小修复是在需要物理去重的观察边界去重，或让 profile 发现/解析独立保留所有受支持别名。不要以要求用户清空或重填原本有效的设置来掩盖回归。新增配置名和跨层迁移都不是必需；保留原单次周期、真实重复未知及已有会话隔离回归。

原四模块本头 137 passed, 5 failed；同输入接回不可变基线 owner 为 134 passed, 8 failed。其中三项新 cadence 回归由红变绿，另五项均在未改动 chat_runtime.py:873 因原 SimpleNamespace fixture 缺 read_only_review 失败，测试身份、位置及异常一致，因果路径与本次 alias 过滤不同，保留为既有检查缺口。它们不是本次 P1 的依据，也没有被声明修好。本地 quick premerge 6直接+2选中检查通过，不能覆盖这些完整调用者反例。作者报告的 Windows 旋转缺口属于另一个 #7222 transport，#7227 URI 修复也不保留 profile 名字；未继承它们的验收。

语义与 CI 对齐

没有新枚举或通用义务，但共享发现结果的名字本身是已有 profile 契约。当前重用 physical Path 身份违反该消费者要求，需要在原 owner 修复并重跑实际保存配置/新会话路径。wait_for_ci=false，本轮没有查询或等待远程 CI。

我的整体评价

存在上述 P1 阻塞，REQUEST_CHANGES。长程影响和用户体验均为 regression：周期读取可用，但已有配置阻止下一次普通会话。架构判断 simplify_now：修复观察去重和逻辑别名选择的边界，无需新框架或用户迁移。 物理目录身份去重属于文件读取的正确性规则；已保存 profile 的别名则是独立的用户选择，不能因另一读者去重而消失。当前全局 codex_homes 同时承担两种含义，现有 profile owner 只按保留路径的名字查找，放置边界需要修正。 保留单物理目录一次心跳观察，同时保留所有已支持 profile 名称的发现和解析；无需迁移、删除用户默认值或新建配置协议。

检查了共享设置 editor 的实际选项和聊天调用链；没有组件、布局、字体或视觉素材改动，视觉构图检查不适用。选项丢失是已复现的功能回归，另列阻塞，不以无布局改动免除。 未独立执行原生Windows junction/旋转、实际已安装App或真实模型；新建会话验证在原生适配器启动处用stub截断，真实配置事务、路径解析与失败前调用完整执行。 已通过、失败及未测试范围分别保留；全树健康、长期效果、实际主机升级与其他 PR 的合入不由本次结论认证。合并仍归维护者，本轮没有合并或安装。

English verdict: REQUEST_CHANGES. Physical-home deduplication fixes cadence but removes valid saved profile aliases from the shared discovery result; all three normal alias forms reproduce failed ordinary conversation opening. Preserve alias-aware profile selection and keep physical deduplication at the appropriate reader boundary.

[loopx-agent]

[P1] Preserve selectable native profile aliases when deduplicating physical homes

codex_homes() is also used by codex_profile_options() and codex_profile_home(), which match the surviving path by name. With CODEX_HOME pointing to .codex-alias -> .codex and a saved codex_chat.codex_profile="codex", this filter removes .codex; the settings option disappears and ChatRuntimeController.open_session() raises The selected Codex profile is unavailable or ambiguous before native startup. The inverse occurs for a saved codex-alias with normal sibling discovery. I reproduced configured/sibling/explicit selection through real directory links, persisted machine-config preview/apply/readback, the shared editor and conversation controller: base resolves/opens, this head fails; a separate profile regression is base 3 passed/head 3 failed. Native adapter startup is stubbed after the failing boundary. Keep physical deduplication scoped to store/automation readers, or retain alias-aware profile discovery/resolution, and add these saved-profile cases alongside the cadence and genuine-duplicate controls.

[mergify]

This pull request has merge conflicts with main and cannot be merged
until they are resolved. Please rebase or merge the base branch, @BigDataDZ.

Choose the remote for the base repository, not an out-of-date fork.
For a fork clone, first inspect git remote -v; upstream must point
to https://github.com/loopx-project/loopx.git. If it is absent, add it
with git remote add upstream https://github.com/loopx-project/loopx.git.
Then run:

git fetch upstream
git rebase upstream/main
# Resolve each conflict, git add the resolved files, then git rebase --continue.
git push --force-with-lease origin HEAD

For a same-repository clone whose origin points to
https://github.com/loopx-project/loopx.git, use origin instead of
upstream for fetch/rebase. If you prefer merging the base, use
git merge <base-remote>/main and push normally.

Keep the DCO Signed-off-by trailer on every commit when you rebase.
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

[BigDataDZ]

Addressed the P1 profile-alias regression in 78b266ec8471f179be5b382bc0d0cec90067e89c.

The previous global physical-home deduplication removed logical profile names used by saved configuration and ordinary conversation startup. Global discovery now preserves baseline logical names/order; physical deduplication is owned solely by the installed-automation reader, including explicit homes= inputs. No configuration reset or migration is required.

Three real Windows junction journeys cover configuration preview/apply/readback, editor options and ordinary open_session: parent discovery 3 passed, original PR discovery 3 failed, repaired discovery 3 passed. Only final native adapter launch is stubbed. Existing native config bytes and saved configuration remain unchanged. Genuine duplicate automations still yield unknown cadence; pausing one restores the interval.

Four owner/consumer suites: 129 passed, 25 skipped. Independent author-side review found no blocker and ran the two changed suites: 88 passed; a fresh primary-agent rerun also passed all 88. Ruff, whitespace, advisory semantic scan and bilingual RFC checks pass. The existing fake adapter now explicitly supplies read_only_review=False; production guards are unchanged.

Known limits remain explicit: companion settings tests have six periodic-report timezone failures identical with parent discovery (missing timezone data); full-suite collection stops at Windows-unavailable fcntl. No full-tree, packaged UI or live Codex execution pass is claimed.

The PR body now reflects the corrected ownership and current evidence. Ready for maintainer re-review on this head; no approval or self-merge claimed.