Code review fixes

[imagekitio]

No description provided.

[Copilot]

Pull request overview

This PR turns the repository into a pnpm workspace monorepo containing the @imagekit/astro SDK package plus a test-app used to exercise the integration across multiple Astro major versions, with Playwright E2E coverage and CI/publish workflows.

Changes:

• Added the @imagekit/astro package implementation: integration, image service (sharp delegation + ImageKit fast-path), components (<Video />, <OgImage />), helpers, and server upload auth helper.
• Added an Astro test app with pages covering Image/Video/OG/Markdown behaviors plus Playwright E2E tests and versioned snapshots.
• Added multi-version test script, pnpm workspace setup, and GitHub Actions CI + publish workflow.

Reviewed changes

Copilot reviewed 56 out of 62 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
test-app/tsconfig.json	Adds strict TS config for the test app.
test-app/src/pages/videos.astro	Video component demo cases for E2E snapshots.
test-app/src/pages/upload.astro	Upload demo UI + client-side upload call.
test-app/src/pages/og.astro	<OgImage> + getOgImageUrl() demo page.
test-app/src/pages/md.md	Markdown image pipeline coverage page.
test-app/src/pages/index.astro	Test app landing page linking to demos.
test-app/src/pages/images.astro	Extensive <Image> behavior matrix for snapshots.
test-app/src/pages/api/upload-auth.ts	API route generating upload auth params (plus CI fallback).
test-app/src/env.d.ts	Astro env typing references for test app.
test-app/README.md	Test app usage docs (run/tests/upload demo).
test-app/playwright.config.ts	Playwright setup + per-Astro-version snapshot directories.
test-app/package.json	Test app dependencies/scripts (Astro, Playwright, adapters).
test-app/e2e/videos.spec.ts	E2E snapshot test for /videos.
test-app/e2e/og.spec.ts	E2E assertions for OG/Twitter meta tags.
test-app/e2e/md.spec.ts	E2E snapshot test for /md.
test-app/e2e/images.spec.ts	E2E snapshot test for /images.
test-app/e2e/snapshot/astro-default/videos.spec.ts/Videos-page-renders-correctly-1.txt	Baseline snapshot for videos page (default matrix entry).
test-app/e2e/snapshot/astro-default/images.spec.ts/Image-page-renders-correctly-1.txt	Baseline snapshot for images page (default matrix entry).
test-app/e2e/snapshot/astro-6/videos.spec.ts/Videos-page-renders-correctly-1.txt	Snapshot for Astro 6 videos rendering.
test-app/e2e/snapshot/astro-6/md.spec.ts/Markdown-page-renders-correctly-1.txt	Snapshot for Astro 6 markdown rendering.
test-app/e2e/snapshot/astro-6/images.spec.ts/Image-page-renders-correctly-1.txt	Snapshot for Astro 6 images rendering.
test-app/e2e/snapshot/astro-5/videos.spec.ts/Videos-page-renders-correctly-1.txt	Snapshot for Astro 5 videos rendering.
test-app/e2e/snapshot/astro-5/md.spec.ts/Markdown-page-renders-correctly-1.txt	Snapshot for Astro 5 markdown rendering.
test-app/e2e/snapshot/astro-5/images.spec.ts/Image-page-renders-correctly-1.txt	Snapshot for Astro 5 images rendering.
test-app/e2e/snapshot/astro-4/videos.spec.ts/Videos-page-renders-correctly-1.txt	Snapshot for Astro 4 videos rendering.
test-app/e2e/snapshot/astro-4/md.spec.ts/Markdown-page-renders-correctly-1.txt	Snapshot for Astro 4 markdown rendering.
test-app/e2e/snapshot/astro-4/images.spec.ts/Image-page-renders-correctly-1.txt	Snapshot for Astro 4 images rendering.
test-app/e2e/snapshot/astro-3/videos.spec.ts/Videos-page-renders-correctly-1.txt	Snapshot for Astro 3 videos rendering.
test-app/e2e/snapshot/astro-3/md.spec.ts/Markdown-page-renders-correctly-1.txt	Snapshot for Astro 3 markdown rendering.
test-app/e2e/snapshot/astro-3/images.spec.ts/Image-page-renders-correctly-1.txt	Snapshot for Astro 3 images rendering.
test-app/astro.config.mjs	Configures ImageKit integration + node adapter + image domains/layout.
test-app/.gitignore	Ignores build artifacts, Playwright outputs, env files.
test-app/.env.example	Example env vars for upload auth demo.
scripts/test-all-versions.sh	Local script to run/update E2E across Astro 3–6 using packed tarball.
README.md	Replaces root README with full SDK documentation and examples.
pnpm-workspace.yaml	Declares workspace packages (imagekit-astro, test-app).
package.json	Adds monorepo root scripts and pnpm packageManager metadata.
LICENSE	Adds MIT license file at repo root.
imagekit-astro/tsup.config.ts	Builds multiple entrypoints (integration/helpers/server/image-service).
imagekit-astro/tsconfig.json	TS config for SDK package build/types.
imagekit-astro/src/types/index.ts	Public prop/type definitions for Video and OG helpers/components.
imagekit-astro/src/services/imagekit-service.ts	Image service implementing IK URL generation + sharp delegation.
imagekit-astro/src/server/uploadAuth.ts	Server-side getUploadAuthParams() implementation.
imagekit-astro/src/server/index.ts	Exports server helper + types.
imagekit-astro/src/lib/imagekit.ts	Resolves ImageKit config from props/virtual module/env.
imagekit-astro/src/integration.ts	Astro integration wiring: image service, remotePatterns/domains, injectTypes.
imagekit-astro/src/helpers/index.ts	getOgImageUrl() helper.
imagekit-astro/src/constants/sizes.ts	OG image default sizes constants.
imagekit-astro/src/components/Video.astro	<Video /> component emitting <video> with IK URL.
imagekit-astro/src/components/OgImage.astro	<OgImage /> component emitting OG/Twitter <meta> tags.
imagekit-astro/src/components/index.ts	Component barrel exports.
imagekit-astro/package.json	Package metadata + exports map for subpath entrypoints.
imagekit-astro/package-lock.json	NPM lockfile committed under SDK package directory.
imagekit-astro/index.ts	SDK root entry exports components/helpers/types.
imagekit-astro/astro.d.ts	Declares *.astro module typings and virtual config module typing.
imagekit-astro/.gitignore	Ignores dist and staged README/LICENSE within package.
.npmrc	pnpm behavior settings for the workspace.
.gitignore	Root ignore rules including tarballs and test artifacts.
.github/workflows/publish.yml	Release-triggered build + npm publish with provenance.
.github/workflows/ci.yml	CI matrix running E2E against Astro 3–6.

Files not reviewed (1)

• imagekit-astro/package-lock.json: Language not supported

---

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

[SwarnimDoegar]

The content collection example given in readme works as expected. But there are some type errors. Type error fixed code. Needs to be updated in imagekit-docs as well:

// src/content.config.ts

import { defineCollection } from 'astro:content';
import { z } from 'astro/zod';
import ImageKit from '@imagekit/nodejs';
import type { Files } from '@imagekit/nodejs/resources/files/files';

const client = new ImageKit({
  privateKey: import.meta.env.IMAGEKIT_PRIVATE_KEY,
});

const gallery = defineCollection({
  loader: async () => {
    const assets = await client.assets.list({
      type: 'file',       // exclude folders
      fileType: 'image',  // only image files
      skip: 0,
      limit: 50,
    });

    return assets
      .filter((asset): asset is Files.File =>
        asset.type === 'file' && !!asset.fileId && !!asset.url
      )
      .map((asset) => ({
        id: asset.fileId ?? '',
        url: asset.url ?? '',
        width: asset.width ?? 0,
        height: asset.height ?? 0,
        name: asset.name ?? '',
        tags: asset.tags ?? [],
      }));
  },
  schema: z.object({
    url: z.string().url(),
    width: z.number(),
    height: z.number(),
    name: z.string(),
    tags: z.array(z.string()),
  }),
});

export const collections = { gallery };