Skip to content

Fix full narration imports and complete project asset lists - #5

Merged
JaysonAlbert merged 2 commits into
mainfrom
codex/local-asset-fixes
Oct 17, 2026
Merged

JaysonAlbert merged 2 commits into
mainfrom
codex/local-asset-fixes

Conversation

@JaysonAlbert

Copy link
Copy Markdown
Owner

Full local narration stems above 15 MiB were rejected using model-reference limits, and project asset lists hid older media after 30 items per type. Local browser/MCP imports now allow audio up to 128 MiB while model references remain capped at 15 MiB. Project-scoped media lists return all project members; global recent lists stay capped at 30 per type.

Also retain the additional project-creation browser regression tests from upstream PR BeatAPI#22. Its implementation is already on this fork's main; no duplicate implementation merge is needed.

Validation:

  • Typecheck, i18n check and workstation production build passed.
  • Node tests: 432 passed, 2 skipped.
  • Isolated Playwright checks: 6 passed, including a 16 MiB WAV upload, 32 assets per media type with project isolation/global limits, and project-creation flows.
  • Tests use an isolated SQLite/media directory and a small synthetic video fixture.

Tracks ONI-128. Existing worktree edits and ignored local data are preserved separately.

Preserve the additional Playwright coverage from the upstream contribution c11b7e3; the corresponding implementation is already on main.
@JaysonAlbert
JaysonAlbert merged commit 8594af6 into main Oct 17, 2026
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.

1 participant