WP-O1: rename authorship tag root from source to origin (PE-client mock)#1815
Open
byteclimber wants to merge 2 commits into
Open
WP-O1: rename authorship tag root from source to origin (PE-client mock)#1815byteclimber wants to merge 2 commits into
byteclimber wants to merge 2 commits into
Conversation
…t source -> origin WP-O1 (#1812, SITES-48000): renames the tag-tree authorship dimension root from `source` to `origin` in the project-engine-client mock fixtures, per the origin-dimension spec (adobe/serenity-docs#46). - mock/seeds.js: DIMENSION_ROOTS.source -> origin, SOURCE_VALUES -> ORIGIN_VALUES, SOURCE_ROOT_TAG_ID -> ORIGIN_ROOT_TAG_ID, sourceHumanTagId -> originHumanTagId in SEED_IDS; dimensionRootTree and both seeded hierarchies updated. Cross-dimension collision fixture (CATEGORY_CHILD_COLLIDING_NAME = 'human') preserved, comments updated to origin. - mock/seeds.js: adds a new `legacy-source-workspace` seed/SEED_IDS entries that keep the PRE-rename `source`-named root (with ai/human children), so downstream repos can test a tolerant resolver that accepts both `origin` and legacy `source` roots during the migration window. Deleted in WP-O6. - mock/factories.js, mock/counterfact/routes/.../aio/tags.js: update doc comments referencing the dimension-root names. - docs/mock-usage.md: documents the fourth seed and the origin rename. - test/mock/seeds.test.js, test/mock/tag-id.test.js, test/mock/tag-view.test.js, test/e2e/project-engine-mock.e2e.js: updated assertions/fixtures for the rename; seeds.test.js adds coverage for the new legacy-source-workspace seed. npm test, npm run test:types, npm run test:e2e, and npm run lint all pass with 100% coverage. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
4 tasks
|
This PR will trigger no release when merged. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Part of the origin-dimension rename: adobe/serenity-docs#46. This is WP-O1 — renames the fully-populated authorship dimension root (
ai/human) fromsourcetooriginin thespacecat-shared-project-engine-clientmock, and adds a legacy pre-rename seed snapshot so downstream repos can test a tolerant resolver against a project still carrying the old name.DIMENSION_ROOTS/SOURCE_VALUES/SOURCE_ROOT_TAG_ID/SEED_IDSrenamed tooriginequivalents (ids re-derive automatically via the runtimetagId(parent, name)hash)human-under-category collision fixturelegacy-source-workspaceseed (pre-rename shape) — to be deleted in a later contract work packagedocs/mock-usage.mdand pinned test suitesTest plan
npm test— 260 passing, 100% coveragenpm run test:e2e— 80 passingnpm run lint— cleannpm run test:types— cleanCloses #1812
Jira: SITES-48000 · Tracking: cross-repo plan in adobe/serenity-docs#46
🤖 Generated with Claude Code