Implement /pet/{petId}/uploadImage persistence and photo URL updates

[Copilot]

POST /pet/{petId}/uploadImage previously returned success without persisting anything or mutating pet data. This change wires the endpoint to store uploaded images under pet-images/{petId} and append the corresponding /photos/{petId}/{imageFileName} entry to the pet’s photoUrls.

• Endpoint behavior

  • Implemented real upload handling in routes/pet/{petId}/uploadImage.ts.
  • Validates pet existence (404) and missing image content (400).
  • Persists image payload to pet-images/{petId}/{imageFileName}.

• Filename resolution

  • Resolves imageFileName from request headers in priority order:
    • x-image-filename
    • x-file-name
    • content-disposition (filename* / filename)
    • fallback: image.bin
  • Uses path.basename(...) to avoid path traversal via header values.

• Pet model mutation

  • Updates in-memory pet state by appending:
    • /photos/{petId}/{imageFileName}
  • Preserves existing photoUrls entries.

• Coverage updates

  • Extended integration flow in test/dummy.test.js to assert:
    • successful upload response
    • photo URL appended on subsequent GET /pet/{petId}
    • file exists at expected disk path
  • Added test cleanup for generated pet-images artifacts.

const imageFileName = getFilenameFromHeaders($.x.headers);
const imageDirectory = path.resolve("pet-images", String($.path.petId));
await mkdir(imageDirectory, { recursive: true });
await writeFile(path.join(imageDirectory, imageFileName), imageContent);

$.context.petsById.set($.path.petId, {
  ...pet,
  photoUrls: [...pet.photoUrls, `/photos/${$.path.petId}/${imageFileName}`],
});

[Copilot]

Pull request overview

This PR makes the Swagger Petstore simulator’s POST /pet/{petId}/uploadImage endpoint actually persist the uploaded image to disk and update the in-memory pet record’s photoUrls accordingly.

Changes:

• Implemented upload handling in routes/pet/{petId}/uploadImage.ts (validate pet existence/body, save file under pet-images/{petId}, append /photos/{petId}/{filename}).
• Extended the integration test to upload an image and assert both the pet mutation and file persistence, including cleanup of generated artifacts.
• Updated package-lock.json to align with the package name and dependency classification.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File	Description
swagger-pet-store/routes/pet/{petId}/uploadImage.ts	Adds image filename resolution, payload extraction, disk persistence, and photoUrls mutation.
swagger-pet-store/test/dummy.test.js	Adds integration coverage for upload + persistence and deletes pet-images after the test run.
swagger-pet-store/package-lock.json	Syncs lockfile metadata/dependency flags with the package configuration.

Files not reviewed (1)

• swagger-pet-store/package-lock.json: Language not supported