Skip to content

Update to latest version #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
SergioNR merged 52 commits into from
Dec 26, 2025
Merged

Update to latest version #28

SergioNR merged 52 commits into from
Dec 26, 2025

Conversation

@SergioNR
Copy link
Member

@SergioNR SergioNR commented Dec 26, 2025

No description provided.

@SergioNR
FIX TYPO in transcription_request_completed example message
5069df0
@SergioNR
FIX TYPO analysisPresignedUploadUrl --> analysisEntryPresignedUploadUrl
20a1b4d
@SergioNR
ADD transcriptionJob model to prisma schema
637f80f
@SergioNR
ADD npm package AWS transcribe
56d9658
@SergioNR
RENAME updateAnalysisEntryInDb to markAnalysisEntryAsSubmitted for cl…
7ec6f1f 
…arity
@SergioNR
ADD transcription normalizer util
6906c00
@SergioNR
ADD functionality to get S3 objects from AWS S3
34a2999
@SergioNR
ADD cronjob to get completedTranscriptionJobs
c9cd86c
@SergioNR
UPDATE prisma schema to include language code
8d410ca
@SergioNR
ADD analysisService layer
48cdbc2
@SergioNR
ADD transcriptionModel - WIP
18e6982
@SergioNR
ADD AWS Transcribe logic
c5002b8
@SergioNR
REFACTOR analysisEntry controller to favor internal functions vs queues
699cda8
@SergioNR
UPDATE swagger & REMOVE "success" attribute from PATCH analysisEntry
fc8df48
@SergioNR
ADD logic to update the transcriptJob after successful AWS transcribe…
a8101ef 
… response
@SergioNR
REMOVE queue-related transcription model functions
42c08ad 
They were used in the microservice - no longer used in the monolith

functions:
- mark as published to queue
- get completed & not published to queue
@SergioNR
REFACTOR transcription model from microservice to monolith
6e9b646
@SergioNR
ADD no-plusplus rule exception to eslint
31725c5
@SergioNR
ADD no-await-in-loop rule exception in eslint
b618c75
@SergioNR
REMOVE lavinMQ implementation
fb3ce07
@SergioNR
ADD error logging to completedTranscriptionJobScheduler CRON job
6a7666d
@SergioNR
FIX typo in transcription job creation function
5bb51e6
@SergioNR
REMOVE mediaType attribute & remove requirement for analysisId
3d5aa70
@SergioNR
ADD logging to transcription request job
610fa72
@SergioNR
ADD logic to handle completed transcription jobs
8b75d68
@SergioNR
ADD no-restricted-syntax rule to eslint exceptions
765db4b
@SergioNR
COMPLETE logic for requesting & processing transcription jobs
aedb52e
@SergioNR
UPDATE db to include transcriptionJob table
8bf9d84
@SergioNR
FIX typo in Transcribe.js
fdd9ef2
@SergioNR
ADD proper error logging to transcription function
e1030ca
SergioNR and others added 22 commits December 22, 2025 15:35
@SergioNR
REFACTOR transcription job retrieval logic for more clarity
0e52887
@SergioNR
ADD prisma schema formatting
db31a5c
@SergioNR
Merge pull request #25 from UXcaptain/refactor-transcription-service-…
cb120eb 
…into-monolith

REFACTOR transcription microservice into the monollith
@SergioNR
ADD config control for programatic transcription enablement
d25395c
@SergioNR
Fix order of cron job scheduler starts
d88d7de
@SergioNR
REMOVE unused code relating to generating presignedurls
13c9d59
@SergioNR
ADD new minio S3 related variables
19a87ca
@SergioNR
UPDATE s3 client config to minIO
af1cbc1
@SergioNR
RENAME AWS_REGION env variable to S3_REGION
876e417
@SergioNR
RENAME AWS_BUCKET env varibale to S3_BUCKET
a87212b
@SergioNR
UPDATE s3 endpoint to be an env variable instead of hardcoded
19d6f37
@SergioNR
Merge pull request #26 from UXcaptain/refactor-aws-s3-to-minio
9ebe505 
REPLACE S3 storage from cloud AWS to self-hosted minIO
@SergioNR
HOTFIX: add missing minIO env variables to compose example
8a5940a
@SergioNR
ADD per-repo docker hub repo
c2b8b7e 
Before this change, all tags were being uploaded to the same repo
@SergioNR
UPDATE docker build & push GH workflow to generate dated tags too
e0303ee 
We want to do this to ensure that rollbacks exist
@SergioNR
ADD temp GH build & push condition to test the workflow
7dea020
@SergioNR
FIX docker metadata-action version
0e0f291
@SergioNR
FIX sha not being generated
0d58478
@SergioNR
REPLACE sha with date
d55e0ba
@SergioNR
FIX date generation
c3a86fe
@SergioNR
REMOVE image generation on non-latest/next branches
4369c03
@SergioNR
Merge pull request #27 from UXcaptain/add-docker-releases
c670f91 
Add dated tagged docker images to be able to rollback
@kiloconnect
Copy link

kiloconnect bot commented Dec 26, 2025

✅ No Issues Found

22 files reviewed | Confidence: 95% | Recommendation: Merge

Review Details

Files: Database schema, AWS integrations, service layer, controllers, cron jobs, Docker configuration, environment setup

Checked: Security, architecture, error handling, database design, service patterns, configuration management

Key Changes Reviewed:

  • ✅ AWS Transcribe integration replacing LavinMQ message broker
  • ✅ New TranscriptionJob database model with proper relationships
  • ✅ Clean service layer architecture with analysisService.js
  • ✅ Proper error handling and logging throughout
  • ✅ MinIO integration for local development
  • ✅ Environment variable standardization (S3_* naming)
  • ✅ Cron job monitoring for transcription status
  • ✅ Docker workflow improvements

Architecture Assessment:
The changes represent a well-designed migration from a message broker-based architecture to a direct AWS service integration. The new approach simplifies the codebase while maintaining reliability through cron-based job monitoring and proper error handling.

@SergioNR SergioNR merged commit 9d33205 into latest Dec 26, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SergioNR