Skip to content

Update AI rules and tighten the db-query skill#892

Merged
tjementum merged 3 commits intomainfrom
pp-1207-update-ai-rules
May 9, 2026
Merged

Update AI rules and tighten the db-query skill#892
tjementum merged 3 commits intomainfrom
pp-1207-update-ai-rules

Conversation

@tjementum
Copy link
Copy Markdown
Member

@tjementum tjementum commented May 9, 2026

Summary & Motivation

  • Backend rule additions:
    • IgnoreQueryFilters calls must name the filter explicitly, e.g. .IgnoreQueryFilters([QueryFilterNames.SoftDelete]), not the bare overload.
    • Database migration filenames must include HHmm, not just yyyyMMdd, so migrations within the same day order deterministically.
    • Forbid #pragma warning disable CS#### — fix the warning instead.
  • Require confirmation in the db-query skill before any mutating SQL (INSERT, UPDATE, DELETE, TRUNCATE, DROP).
  • Use the explicit branch name when setting upstream after gh pr creategit push -u origin HEAD doesn't reliably configure tracking.

Checklist

  • I have added tests, or done manual regression tests
  • I have updated the documentation, if necessary

@tjementum tjementum added the Enhancement New feature or request label May 9, 2026
@tjementum tjementum self-assigned this May 9, 2026
@linear
Copy link
Copy Markdown

linear Bot commented May 9, 2026

PP-1207

@tjementum tjementum moved this to 🏗 In Progress in Kanban board May 9, 2026
@tjementum tjementum force-pushed the pp-1207-update-ai-rules branch from d5cf521 to 1d556a8 Compare May 9, 2026 22:48
@tjementum tjementum merged commit b21867b into main May 9, 2026
2 checks passed
@tjementum tjementum deleted the pp-1207-update-ai-rules branch May 9, 2026 22:49
@github-project-automation github-project-automation Bot moved this from 🏗 In Progress to ✅ Done in Kanban board May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant