Skip to content

fix: remove unused uuid dependency and update scenario creation logic#6

Merged
pikann merged 1 commit into
masterfrom
fix/fix-duplicated-uuid-error
Jun 17, 2026
Merged

fix: remove unused uuid dependency and update scenario creation logic#6
pikann merged 1 commit into
masterfrom
fix/fix-duplicated-uuid-error

Conversation

@pikann

@pikann pikann commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes the github.com/google/uuid dependency (go.mod / go.sum) — ID generation is now delegated to PostgreSQL.
  • Refactors createScenario to use INSERT … RETURNING id (via db.Query) instead of generating a UUID client-side and passing it as $1.
  • Fixes a duplicated path segment in plugin.json: /projects/:projectId/projects/:projectId/tasks/…/projects/:projectId/tasks/….

Test plan

  • Create a BDD scenario via the API and confirm the response contains the database-generated id.
  • Verify the PATCH route (/projects/:projectId/tasks/:taskId/bdd-scenarios/:scenarioId) resolves correctly after the path fix.
  • Confirm go.mod no longer lists github.com/google/uuid.

@pikann pikann merged commit 7e946ba into master Jun 17, 2026
3 checks passed
@pikann pikann deleted the fix/fix-duplicated-uuid-error branch June 17, 2026 06:50
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.

1 participant