fix: preserve seed data _id as stable record id across page refreshes

[Copilot]

Detail page shows "Record not found" on browser refresh because the in-memory MSW kernel reboots with new timestamp-based IDs, invalidating the ID in the URL.

Root cause

syncDriverIds() unconditionally overwrites seed data's stable _id (e.g. "101") with the driver-generated id (e.g. "opportunity-1772422975850-6"). On refresh, a new kernel generates different timestamps → old URL IDs are orphaned.

// Before: always overwrites seed _id
record._id = record.id;

// After: promotes seed _id to canonical id when present
if (record._id != null && record._id !== record.id) {
  record.id = record._id;
} else if (record.id) {
  record._id = record.id;
}

Changes

• apps/console/src/mocks/createKernel.ts — syncDriverIds now promotes explicit seed _id to id, keeping record identifiers stable across kernel reboots
• packages/data-objectstack/src/index.ts — Defensive array handling in findOne $expand path, consistent with normalizeQueryResult for find()
• apps/console/src/__tests__/MSWServer.test.tsx — Tests verifying seed IDs are preserved and records are fetchable by stable _id via HTTP

Original prompt

https://demo.objectui.org/apps/crm_app/opportunity/record/opportunity-1772422975850-6 进入详情页刷新显示record not found

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
objectui	Ready	Preview, Comment	Mar 2, 2026 4:21am
objectui-demo	Ready	Preview, Comment	Mar 2, 2026 4:21am
objectui-storybook	Ready	Preview, Comment	Mar 2, 2026 4:21am

[Copilot]

Pull request overview

Fixes unstable record URLs in the console MSW/in-memory mock environment by ensuring seed _id values remain the canonical identifier across kernel reboots (page refresh), preventing “Record not found” deep-link failures.

Changes:

• Update syncDriverIds() to promote explicit seed _id → id (and otherwise derive _id from id) for stable identifiers across refreshes.
• Make ObjectStackAdapter.findOne() with $expand robust to servers returning a flat array response.
• Add MSW integration tests to verify stable seed IDs and HTTP fetch-by-id behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
apps/console/src/mocks/createKernel.ts	Adjusts seed record ID synchronization to preserve stable _id values across MSW kernel restarts.
packages/data-objectstack/src/index.ts	Adds defensive handling for array-shaped responses in the $expand path of findOne().
apps/console/src/tests/MSWServer.test.tsx	Adds integration coverage ensuring seed IDs are preserved and records are retrievable by stable ID over HTTP.

[Copilot]

The new array-response handling in the findOne() $expand path isn’t covered by existing adapter tests. Please add/extend a unit test (e.g. in src/expand.test.ts) where the raw fetch returns an unwrapped array (or { success: true, data: [...] }) to ensure findOne() returns the first record instead of null.

Suggested change

	const resultObj = result as { records?: T[]; value?: T[] };
	const records = resultObj.records || resultObj.value || [];
	const resultObj = result as { records?: T[]; value?: T[]; data?: T[] };
	const records = resultObj.records || resultObj.value || resultObj.data || [];