Skip to content

docs(agents): consolidate durable engineering notes into scoped AGENTS.md#646

Merged
CybotTM merged 1 commit into
mainfrom
docs/agents-md-consolidate-notes
Jul 19, 2026
Merged

docs(agents): consolidate durable engineering notes into scoped AGENTS.md#646
CybotTM merged 1 commit into
mainfrom
docs/agents-md-consolidate-notes

Conversation

@CybotTM

@CybotTM CybotTM commented Jul 18, 2026

Copy link
Copy Markdown
Member

Consolidates durable, verified engineering notes into the four scoped AGENTS.md files. Docs-only, no code changes.

Each fact was checked against the codebase before writing; several stale or incorrect drafts were dropped or corrected — e.g. the API is /api/v2/* with a {error, message} shape (not Nelmio / /api/v1 / RFC 9457), the ExtJS→SolidJS migration is already complete (so no "ongoing" note), and the merge gate does include the required CI Success aggregate check.

  • frontend/AGENTS.md — Ark UI date-picker / inline-editor focus pattern, "Philosophy A" grid-editing architecture, stack decisions (Ark UI vs Kobalte, Tailwind 4, no Vue/Svelte), vitest-axe/jsdom note.
  • e2e/AGENTS.mddb-e2e reseed procedure, image bind-mount rebuild flow, TestClock vs APP_FROZEN_TIME, usable test users, save-round-trip timeout guidance.
  • tests/AGENTS.md — portable LIKE … ESCAPE, logInSession() EM identity-map validator gotcha.
  • src/AGENTS.mdSaveDto grandfathering pattern, Holiday DBAL-only entity, self-migrating prod entrypoint, and a new "Deployment, CI & merge (operational)" section.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the documentation in several AGENTS.md files across the repository (e2e, frontend, src, and tests). The updates provide detailed instructions and architectural guidelines on database seeding, rebuilding after code changes, test environment configurations, frontend stack decisions (such as Ark UI and SolidJS), custom date picker implementations, backend validation rules, DBAL usage for specific entities, CI/CD deployment workflows, and testing best practices (including SQLite compatibility and functional test limitations). There are no review comments, so I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

…S.md

Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
@CybotTM
CybotTM force-pushed the docs/agents-md-consolidate-notes branch from 31c412e to abc2beb Compare July 19, 2026 16:52
@CybotTM
CybotTM marked this pull request as ready for review July 19, 2026 16:52
Copilot AI review requested due to automatic review settings July 19, 2026 16:52
@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@CybotTM
CybotTM merged commit 8436581 into main Jul 19, 2026
17 of 18 checks passed
@CybotTM
CybotTM deleted the docs/agents-md-consolidate-notes branch July 19, 2026 17:02
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