Support media gallery

[freekh]

No description provided.

[Copilot]

Pull request overview

This PR adds media gallery awareness to the Val VS Code extension/language server, including diagnostics + quick-fixes for unregistered gallery items and improved path completions scoped to a referenced gallery.

Changes:

• Add server-side validation to detect when c.image/c.file references a media gallery but the referenced asset is not registered or not in the gallery directory.
• Add client quick-fixes to (a) insert a metadata entry into the gallery module and (b) move an asset into the configured gallery directory and update references.
• Improve completion providers by resolving the schema at the cursor’s field-path (including richtext href) and filtering image/file path completions to the referenced gallery directory.

Reviewed changes

Copilot reviewed 14 out of 17 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
server/src/server.ts	Runs new media gallery validation and emits LSP diagnostics with fix data.
server/src/metadataUtils.ts	Adjusts image-size input type for updated dependency behavior.
server/src/metadataUtils.test.ts	Updates tests to match new image-size input handling.
server/src/mediaGalleryValidation.ts	Implements (source,schema) walker to produce media gallery diagnostics.
server/src/mediaGalleryValidation.test.ts	Adds tests for the media gallery diagnostics walker.
server/src/completionProviders.ts	Uses field-path schema resolution; filters file/image completions to referenced gallery directory.
server/src/completionFieldSchema.ts	New helpers to derive field paths from AST and resolve schema at that path (incl. richtext href).
server/src/completionFieldSchema.test.ts	Adds unit tests for schema resolution helpers.
server/package.json	Bumps @valbuild/* dependencies to versions that support media gallery behavior.
server/package-lock.json	Lockfile updates from dependency bumps (incl. transitive deps).
package-lock.json	Root lockfile version bump.
client/src/extension.ts	Registers new commands and adds quick-fix actions for media gallery diagnostics.
client/src/commands/moveFileToGalleryDirectory.ts	New command to move an asset into the gallery directory and update the reference.
client/src/commands/addToMediaGallery.ts	New command to insert a gallery entry with inferred metadata into the referenced gallery module.
client/src/commands/addToMediaGallery.test.ts	Unit tests for helper functions used by the gallery insertion command.
client/package.json	Bumps @valbuild/core to match server/core expectations.
client/package-lock.json	Lockfile updates for client dependency bump.

Files not reviewed (2)

• client/package-lock.json: Language not supported
• server/package-lock.json: Language not supported

Comments suppressed due to low confidence (1)

client/src/commands/addToMediaGallery.ts:177

• getImageMetadata/getFileMetadata ultimately call path.join(rootPath, relativePath) (client/src/metadataUtils.ts). If data.path is a Val-style path starting with /public/..., path.join treats it as absolute and ignores rootPath, so metadata lookup will likely fail. Consider stripping the leading / (for FS resolution only) or updating metadataUtils to treat leading-slash Val paths as project-root relative.

  const mediaType = data.mediaType ?? "images";
  const rawMetadata =
    mediaType === "images"
      ? getImageMetadata(data.path, uri)
      : getFileMetadata(data.path, uri);
  const metadata = stripAlt(rawMetadata as Record<string, string | number>);
  if (Object.keys(metadata).length === 0) {
    vscode.window.showErrorMessage(
      `Could not read metadata for ${data.path}. Make sure the file exists at ${path.join(valRoot, ...data.path.split("/"))}.`,
    );

---

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