Skip to content

Conversation

@eoinsha
Copy link
Collaborator

@eoinsha eoinsha commented Dec 19, 2025

  • chore: add second function for manual scaling
  • feat: add API for auto and processing for manual capacity providers

@lmammino lmammino requested a review from Copilot January 7, 2026 15:24
@lmammino lmammino marked this pull request as ready for review January 7, 2026 15:25
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 transforms a minimal Lambda MI example into a comprehensive video processing service demonstration. It adds a dual-capacity provider architecture with auto-scaling for API workloads and manual scaling for batch processing, orchestrated through Step Functions.

  • Replaces Jest testing framework with Biome for linting/formatting
  • Implements a complete video processing API with DynamoDB storage
  • Adds Step Functions workflow for dynamic capacity scaling
  • Creates three Lambda functions: API handler, video processor, and scaling controller

Reviewed changes

Copilot reviewed 15 out of 22 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tsconfig.json Simplified formatting and added skipLibCheck option
package.json Removed Jest dependencies, added Biome for code quality
package-lock.json Updated dependencies reflecting Jest removal and Biome addition
test/lambda-mi.test.ts Removed placeholder test file
jest.config.js Removed Jest configuration
lib/lambda-mi-stack.ts Expanded from basic example to full video processing service with dual capacity providers
functions/video-api/* New API Lambda function for video CRUD operations
functions/video-processor/* New processor Lambda for simulated video processing
functions/scaling-function/* New scaling controller Lambda for manual capacity adjustments
biome.json New Biome configuration for linting and formatting
demo.sh New demo script for testing the deployed service
README.md Comprehensive documentation of the video processing architecture
.github/copilot-instructions.md New GitHub Copilot guidance document
cdk.context.json Added VPC lookup context
Files not reviewed (3)
  • functions/scaling-function/package-lock.json: Language not supported
  • functions/video-api/package-lock.json: Language not supported
  • functions/video-processor/package-lock.json: Language not supported

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

@lmammino lmammino merged commit ab102fa into main Jan 8, 2026
6 checks passed
@lmammino lmammino deleted the mixed-capacity-providers branch January 8, 2026 10:16
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.

3 participants