Skip to content

Remove field-spec instances from packages/base#4724

Open
richardhjtan wants to merge 1 commit intomainfrom
CS-11032-remove-base-specs
Open

Remove field-spec instances from packages/base#4724
richardhjtan wants to merge 1 commit intomainfrom
CS-11032-remove-base-specs

Conversation

@richardhjtan
Copy link
Copy Markdown
Contributor

The field-spec JSON files under packages/base/fields/ are moving to boxel-catalog.

Three host tests had hard-coded selectors pointing at the old base field-spec URLs. They now use inline fixtures in their test realms, making them self-contained.

There is a test to verify that the base specs exist in boxel-catalog: cardstack/boxel-catalog#557

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Preview deployments

Host Test Results

    1 files  ±0      1 suites  ±0   1h 59m 17s ⏱️ - 4m 13s
2 644 tests +1  2 629 ✅ +1  15 💤 ±0  0 ❌ ±0 
2 663 runs  +1  2 648 ✅ +1  15 💤 ±0  0 ❌ ±0 

Results for commit 9029ab5. ± Comparison against earlier commit 1a42cd3.

Realm Server Test Results

    1 files  ±0      1 suites  ±0   15m 54s ⏱️ -54s
1 297 tests ±0  1 297 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 376 runs  ±0  1 376 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 9029ab5. ± Comparison against earlier commit 1a42cd3.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the field-spec JSON instances from packages/base/fields/ (they’re moving to boxel-catalog) and updates host UI/tests to stop relying on the old https://cardstack.com/base/fields/* URLs by using inline fixtures and a configurable default field spec ID.

Changes:

  • Removed field-spec instance JSON files from packages/base/fields/.
  • Updated host runtime config to expose defaultFieldSpecId, and updated the “Create File” modal to use it for Field Definition defaults.
  • Updated integration/acceptance tests to use inline field-spec fixtures in their test realms and updated selectors to use testRealmURL.

Reviewed changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/host/tests/integration/components/operator-mode/setup.gts Adds an inline fields/biginteger-field.json fixture to the integration test realm.
packages/host/tests/integration/components/operator-mode-links-test.gts Updates card-catalog selectors to use the test realm URL for the field spec.
packages/host/tests/acceptance/code-submode/schema-editor-test.ts Adds inline field-spec fixtures (BigInteger/Date) and updates selectors to use the test realm URL.
packages/host/tests/acceptance/code-submode/create-file-test.gts Adds inline fixtures for fields/biginteger-field.json and fields/field.json; updates selectors to use the test realm URL.
packages/host/config/environment.js Introduces ENV.defaultFieldSpecId (test hardcode; computed from catalog realm when available).
packages/host/app/config/environment.ts Extends the typed config shape with defaultFieldSpecId?: string.
packages/host/app/components/operator-mode/create-file-modal.gts Uses config.defaultFieldSpecId as the default spec for Field Definition creation.
packages/base/fields/website-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/url-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/text-area-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/string-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/realm-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/phone-number.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/percentage-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/number-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/markdown-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/ethereum-address-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/email-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/datetime-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/date-range-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/date-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/country-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/command-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/color-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/code-ref-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/boolean-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/biginteger-field.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/base64-image.json Removed base field spec instance JSON (moved to catalog).
packages/base/fields/address-field.json Removed base field spec instance JSON (moved to catalog).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/host/app/components/operator-mode/create-file-modal.gts Outdated
Comment thread packages/host/config/environment.js
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 30 out of 30 changed files in this pull request and generated 2 comments.

Comment thread packages/host/app/components/operator-mode/create-file-modal.gts
Comment thread packages/host/config/environment.js
@richardhjtan richardhjtan force-pushed the CS-11032-remove-base-specs branch 3 times, most recently from 31e26c1 to 4eb2ed9 Compare May 8, 2026 12:13
@richardhjtan richardhjtan requested a review from Copilot May 8, 2026 12:14
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread packages/host/config/environment.js
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread packages/host/config/environment.js
@richardhjtan richardhjtan marked this pull request as ready for review May 8, 2026 12:48
@richardhjtan richardhjtan requested review from a team May 8, 2026 12:48
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1a42cd3eee

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/host/config/environment.js
@habdelra
Copy link
Copy Markdown
Contributor

habdelra commented May 8, 2026

Probably you want to link this to the PR that adds them in catalog. the merge for this which does the staging deploy and the production deploy for this absolutely need to be coordinated with their addition to the catalog realm in each environment, or it will be a very chaotic transition. cardstack/boxel-catalog#557

@richardhjtan
Copy link
Copy Markdown
Contributor Author

Probably you want to link this to the PR that adds them in catalog. the merge for this which does the staging deploy and the production deploy for this absolutely need to be coordinated with their addition to the catalog realm in each environment, or it will be a very chaotic transition. cardstack/boxel-catalog#557

Agreed with the measure, thank you @habdelra

The field-spec JSON files under packages/base/fields/ are moving
to boxel-catalog

Three host tests had hard-coded selectors pointing at the old base
field-spec URLs. They now use inline fixtures in their test realms,
making them self-contained.
@richardhjtan richardhjtan force-pushed the CS-11032-remove-base-specs branch from 1a42cd3 to 9029ab5 Compare May 11, 2026 04:09
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.

3 participants