QRun-IO/qbit-quick-search
3
0 Fork 0
Code Issues 2 Pull Requests 4 Actions Packages Projects Releases 14 Wiki Activity

Every quick_search table explains itself; guardrail and primer tests #58

Merged
KofTwentyTwo merged 1 commits from feature/tables-populated-defaults into develop 2026-10-14 22:19:52 +00:00
Conversation 0 Commits 1 Files Changed 8
+702
KofTwentyTwo commented 2026-10-14 21:52:48 +00:00 (Migrated from github.com)
Copy link
Copy source

Audit

All 8 tables are read/written by runtime code. Config tables are seeded at runtime by QuickSearchConfigPrimer (this qbit ships no DDL or changelog), so no changelog was added. The primer creates the quick_search_settings row (id 1) when missing, even with no code-declared tables (verified, now tested). The home already shows status, so no Setup card was added.

Changes

  • help/quick-search.yaml (app + 8 tables, each with "How rows get created") applied by QuickSearchHelp (STAND-IN core#1198) at the end of produce; only fills empty slots.
  • Existing withHelpContent on realTimeErrorCount untouched.
  • Docs: README, CHANGELOG, docs/MIGRATION-1.1.md (no UPGRADE.md exists).

Tests

QuickSearchTablesGuardrailTest, QuickSearchSeededDefaultsTest (fresh memory backend: populate, idempotent, edit survives, settings row with no code tables). CI=true ./mvnw verify green.

Not done

None.

🤖 Generated with Claude Code

## Audit All 8 tables are read/written by runtime code. Config tables are seeded at runtime by `QuickSearchConfigPrimer` (this qbit ships no DDL or changelog), so no changelog was added. The primer creates the `quick_search_settings` row (id 1) when missing, even with no code-declared tables (verified, now tested). The home already shows status, so no Setup card was added. ## Changes - `help/quick-search.yaml` (app + 8 tables, each with "How rows get created") applied by `QuickSearchHelp` (STAND-IN core#1198) at the end of `produce`; only fills empty slots. - Existing `withHelpContent` on `realTimeErrorCount` untouched. - Docs: README, CHANGELOG, docs/MIGRATION-1.1.md (no UPGRADE.md exists). ## Tests `QuickSearchTablesGuardrailTest`, `QuickSearchSeededDefaultsTest` (fresh memory backend: populate, idempotent, edit survives, settings row with no code tables). `CI=true ./mvnw verify` green. ## Not done None. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
architecture
Architecture review: coupling, cohesion, UI API
 bug
Something isn't working
 dependencies
documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 github_actions
Pull requests that update GitHub Actions code
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 java
Pull requests that update java code
 question
Further information is requested
 release:rc
Merging this develop PR cuts the next release candidate
 secuurity
wontfix
This will not be worked on
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
forge-recovery (Forge Recovery) james.maes (James Maes)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: QRun-IO/qbit-quick-search#58