Skip to content

fix: Remove duplicate workflow causing double issue creation - #13

Merged
paperdiamond merged 3 commits into
mainfrom
fix/duplicate-issues
Dec 8, 2025
Merged

paperdiamond merged 3 commits into
mainfrom
fix/duplicate-issues

Conversation

@paperdiamond

Copy link
Copy Markdown
Owner

Summary

  • Root cause: Two workflow files (gitlin.yml and gitlin-bot.yml) both triggered on /create-issues comments, causing GitHub to run BOTH workflows and create duplicate issues
  • Removes gitlin-bot.yml (the older, simpler version)
  • Adds concurrency control to prevent race conditions from webhook retries

Test plan

  • Post /create-issues on this PR to verify only one set of issues is created
  • Check GitHub Actions to verify only one workflow runs

The root cause of duplicate issue creation was two workflow files
(.github/workflows/gitlin.yml and .github/workflows/gitlin-bot.yml)
both triggering on the same events. When `/create-issues` was posted,
GitHub triggered BOTH workflows simultaneously, creating 2x issues.

Changes:
- Remove gitlin-bot.yml (duplicate, simpler version)
- Add concurrency control to prevent race conditions from webhook retries
@paperdiamond

Copy link
Copy Markdown
Owner Author

Found a potential SQL injection vulnerability in the user search endpoint. The searchUsers function in api/users.ts directly interpolates user input into the query string without sanitization. An attacker could potentially extract or modify database contents. This needs to be addressed before we can deploy to production.

@paperdiamond

Copy link
Copy Markdown
Owner Author

The checkout flow is broken on mobile devices. When users tap 'Place Order', the button doesn't respond and they can't complete their purchase. We're getting customer complaints and this is affecting revenue. Tested on iOS Safari and Chrome Android - both have the issue.

@paperdiamond

Copy link
Copy Markdown
Owner Author

The dashboard API response time has increased from 200ms to 1.2s after the last deployment. Looking at the logs, it seems the getAnalytics function is making redundant database calls. We should add caching or optimize the query to reduce load times.

@paperdiamond

Copy link
Copy Markdown
Owner Author

It would be nice to add dark mode support to the settings page. Some users have requested this feature. We could use CSS variables and add a toggle in user preferences. Not blocking anything but would improve the user experience for night-time usage.

@paperdiamond

Copy link
Copy Markdown
Owner Author

/create-issues

@github-actions

github-actions Bot commented Dec 8, 2025

Copy link
Copy Markdown

✅ Created 4 Linear issues:

  • GIT-146 Fix SQL injection vulnerability in user search endpoint
  • GIT-147 Fix non-responsive checkout button on mobile devices
  • GIT-148 Optimize dashboard API performance and reduce response time
  • GIT-149 Add dark mode support to settings page

@paperdiamond
paperdiamond merged commit 9dddd06 into main Dec 8, 2025
1 check passed
@paperdiamond
paperdiamond deleted the fix/duplicate-issues branch December 8, 2025 05:03
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