Skip to content

Prod fix#68

Merged
NickSavino merged 19 commits into
mainfrom
development
Mar 14, 2026
Merged

Prod fix#68
NickSavino merged 19 commits into
mainfrom
development

Conversation

@NickSavino

Copy link
Copy Markdown
Contributor

No description provided.

NickSavino and others added 19 commits February 4, 2026 03:35
Syncing main to development
Feat/middleware-wiring-webhook-setup
* added user calibration endpoints and schema"
- Added user_calibration migration
- Added sqlc queries for calibration CRUD operations
- added /api/users/me/calibration endpoints: GET, POST, DELETE
- me endpoint now fetches user from clerk id using middleware, enhancing security
- removed GET methods for user (GET /{id}, GET /)

* Fixed line endings

* Fixed time formatting in handler

* Added NOT NULL constrain to updated_at, created_at in db table

switch go dev container port to correct mapping (8080)

fixed variable typo in middleware_auth.go

---------

Co-authored-by: NicolaSavino <nick.savino@arcurve.com>
Co-authored-by: NicolaSavino <nick.savino@arcurve.com>
* Implemented workout and calibration schema with included sqlc queries. Updated makefile and seeding files

* Regenerated SQLC to match new queries and models (workout type + workout session)

* Added workout session service

* Added Workout Handler Endpoint

* Addressed PR comments
- fixed syntax error in makefile
-  Fixed migration schema
- fixed seeding script
- added /service/helps/helpers.util file
- renamed Dtos to match go standards, (Id -> ID)

* Updated API endpoints

* updated dtos to match casing convention. updated workout_session logic

* Added functionality for workout sets and reps. (services, handlers, migrations, queries)

* Addressed PR comments

* Addressed PR comments

---------

Co-authored-by: NicolaSavino <nick.savino@arcurve.com>
* fix: added clerk webhook secret

* updated CRLF line endings to LF
* fix: added clerk webhook secret

* updated CRLF line endings to LF

* fix
…pment

# Conflicts:
#	scripts/ci/deploy_via_ssm.sh
@NickSavino
NickSavino merged commit 82d1ca7 into main Mar 14, 2026
@qodo-code-review

Copy link
Copy Markdown

Review Summary by Qodo

Normalize shell script line endings with gitattributes

📦 Other

Grey Divider

Walkthroughs

Description
• Normalize line endings in shell script to LF
• Add Git attributes configuration for consistent line endings
Diagram
flowchart LR
  A["deploy_via_ssm.sh<br/>CRLF line endings"] -- "normalize to LF" --> B["deploy_via_ssm.sh<br/>LF line endings"]
  C["Add .gitattributes"] -- "enforce LF for *.sh" --> D["Consistent shell scripts"]
Loading

Grey Divider

File Changes

1. scripts/ci/deploy_via_ssm.sh Formatting +37/-37

Normalize line endings to LF

• Normalized line endings from CRLF to LF throughout the file
• No functional code changes, only line ending normalization

scripts/ci/deploy_via_ssm.sh


2. .gitattributes ⚙️ Configuration changes +1/-0

Add gitattributes for shell script line endings

• Added new .gitattributes file to enforce LF line endings
• Configured all shell scripts (*.sh) to use LF line endings
• Prevents line ending inconsistencies across different operating systems

.gitattributes


Grey Divider

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented Mar 14, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider


Remediation recommended

1. No newline in gitattributes 🐞 Bug ⛯ Reliability
Description
.gitattributes is committed without a trailing newline, which Git flags and can cause edge-case
parsing/patching issues for tools that assume newline-terminated lines.
Code

.gitattributes[1]

+*.sh text eol=lf
Evidence
The file consists of a single attribute line and the PR diff explicitly marks it as missing an
end-of-file newline; keeping this file newline-terminated avoids tooling edge cases when applying
patches or processing attributes.

.gitattributes[1-1]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`.gitattributes` is missing a trailing newline at end-of-file.

### Issue Context
Git diff shows `\ No newline at end of file`, which can create avoidable edge cases with text-processing and patch tooling.

### Fix Focus Areas
- .gitattributes[1-1]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

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