Skip to content

Conversation

@kevalyq
Copy link
Contributor

@kevalyq kevalyq commented Nov 16, 2025

Description

Add OpenAPI 3.1 documentation for the Secret Attachments API (implemented in SecPal/api#177).

Related Issues

Changes

  • Added /api/v1/secrets/{secretId}/attachments endpoints (POST, GET)
  • Defined SecretAttachment schema
  • Added file upload multipart/form-data examples
  • Added response schemas and error cases

Checklist

  • OpenAPI spec validates (npm run lint)
  • Examples provided for all endpoints
  • Error responses documented
  • REUSE compliant
  • Preflight script passes

- POST /secrets/{secret}/attachments (upload)
- GET /secrets/{secret}/attachments (list)
- GET /attachments/{attachment}/download (download)
- DELETE /attachments/{attachment} (delete)
- Added SecretAttachment schema
- Includes authentication, validation, and error responses

Related: #175
Copilot AI review requested due to automatic review settings November 16, 2025 16:34
@github-actions
Copy link

💡 Tip: Consider Using Draft PRs

Benefits of opening PRs as drafts initially:

  • 💰 Saves CI runtime and Copilot review credits
  • 🎯 Automatically sets linked issues to "🚧 In Progress" status
  • 🚀 Mark "Ready for review" when done to trigger full CI pipeline

How to convert:

  1. Click "Still in progress? Convert to draft" in the sidebar, OR
  2. Use gh pr ready when ready for review

This is just a friendly reminder - feel free to continue as is! 😊

Copilot finished reviewing on behalf of kevalyq November 16, 2025 16:36
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds OpenAPI 3.1 documentation for the Secret Attachments API, providing a contract for file attachment operations on secrets. The documentation covers the complete CRUD lifecycle for attachments including upload, list, download, and delete operations.

Key changes:

  • Defines SecretAttachment schema with file metadata fields
  • Adds four new endpoints under the "Secret Attachments" tag
  • Documents multipart/form-data file upload with binary format

@kevalyq kevalyq merged commit 0fbf52e into main Nov 16, 2025
22 checks passed
@kevalyq kevalyq deleted the docs/secret-attachments-openapi branch November 16, 2025 16:37
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