Skip to content

WEBDEV-8509: Migrate iaux-item-metadata into elements#53

Open
jbuckner wants to merge 3 commits into
WEBDEV-8507-migrate-field-parsersfrom
WEBDEV-8509-migrate-item-metadata
Open

WEBDEV-8509: Migrate iaux-item-metadata into elements#53
jbuckner wants to merge 3 commits into
WEBDEV-8507-migrate-field-parsersfrom
WEBDEV-8509-migrate-item-metadata

Conversation

@jbuckner
Copy link
Copy Markdown
Collaborator

@jbuckner jbuckner commented May 27, 2026

Move @internetarchive/iaux-item-metadata into src/models/item-metadata/ per WEBDEV-8506 (#50): models (File, Metadata, Review, SpeechMusicASREntry, Task) + 9 typed field wrappers.

Stacked on #51 (field-parsers must be in tree first); base auto-rebases to main when #51 merges.

  • Imports rewritten to the in-tree @src/parsers/... paths (barrel → one explicit import per parser — clearer + tree-shake-friendly).

No runtime change; build + tests green. QA-waived (code move).

WEBDEV-8509

Moves @internetarchive/iaux-item-metadata models into
src/models/item-metadata/. Field-parser imports rewritten to
@src/parsers/* (the in-tree paths landed by WEBDEV-8507). Adds
typescript-memoize as a runtime dependency.

The addeddate test was using local-time setters to compare against
a UTC-suffixed ISO timestamp; switched to setUTCHours/etc so the
test isn't TZ-dependent under Vitest's browser provider.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 27, 2026

PR Preview Action v1.8.1

🚀 View preview at
https://internetarchive.github.io/elements/pr/pr-53/

Built to branch ghpages at 2026-05-28 21:00 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

jbuckner and others added 2 commits May 28, 2026 13:59
Add an interactive `item-metadata-story` built on the shared
`story-template`. Edit a raw archive.org metadata JSON record and watch
the `Metadata` model lazily cast each field to its typed `MetadataField`
(Dates, numbers, byte counts, normalized multi-value arrays). Broaden the
demo `app-root.ts` glob to `../src/**/*-story.ts` so stories outside
`elements`/`labs` are auto-discovered.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copy link
Copy Markdown

@bfalling bfalling left a comment

Choose a reason for hiding this comment

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

LGTM

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