Compare commits
18 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 3b6c5297f6 | |||
| a24924c230 | |||
| 11826356c1 | |||
| e18b1df744 | |||
| 595c172900 | |||
| cf225826c0 | |||
| 25c05bc2d8 | |||
| 4ec6295d1d | |||
| 2d18478110 | |||
| 8b95f10c65 | |||
| f3d96fd700 | |||
| 4762f6c0c3 | |||
| 7117a93376 | |||
| 085e34f062 | |||
| 32c388760b | |||
| 98b0bba398 | |||
| 9836e87392 | |||
| 2579539084 |
@@ -0,0 +1,88 @@
|
|||||||
|
# Create Defect in YouTrack
|
||||||
|
|
||||||
|
Automated defect creation workflow. Topic/hint: `$ARGUMENTS`
|
||||||
|
|
||||||
|
## Step 1 — Gather Context
|
||||||
|
|
||||||
|
Use `AskUserQuestion` tool to ask the user (max 4 questions at once):
|
||||||
|
|
||||||
|
1. **Component** — Where does the bug occur? (e.g. move generation, FEN parsing, castling, UI, API)
|
||||||
|
2. **What breaks** — What is the actual (broken) behavior?
|
||||||
|
3. **Expected** — What should happen instead?
|
||||||
|
4. **Reproducibility** — Is it always reproducible? Any known trigger conditions?
|
||||||
|
|
||||||
|
If `$ARGUMENTS` already answers some of these, skip those questions.
|
||||||
|
|
||||||
|
## Step 2 — Research (if needed)
|
||||||
|
|
||||||
|
If the bug involves domain logic or rules:
|
||||||
|
- Search repo for relevant code (`Grep`/`Bash`).
|
||||||
|
- Check test files for existing coverage of the broken area.
|
||||||
|
- Do NOT guess at root cause. Surface findings before drafting.
|
||||||
|
|
||||||
|
## Step 3 — Draft Defect
|
||||||
|
|
||||||
|
Compose the full defect report using this template:
|
||||||
|
|
||||||
|
```
|
||||||
|
Summary
|
||||||
|
|
||||||
|
[One-sentence description of what is broken.]
|
||||||
|
|
||||||
|
|
||||||
|
Steps to Reproduce
|
||||||
|
|
||||||
|
1. Step one
|
||||||
|
2. Step two
|
||||||
|
3. Step three
|
||||||
|
|
||||||
|
|
||||||
|
Expected Behavior
|
||||||
|
|
||||||
|
[What should happen.]
|
||||||
|
|
||||||
|
|
||||||
|
Actual Behavior
|
||||||
|
|
||||||
|
[What actually happens.]
|
||||||
|
|
||||||
|
|
||||||
|
Environment / Notes
|
||||||
|
|
||||||
|
[Any relevant context: FEN positions, game conditions, config, browser, OS — only if applicable.]
|
||||||
|
```
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Steps must be minimal and reproducible.
|
||||||
|
- Expected vs actual: concrete and unambiguous.
|
||||||
|
- Omit "Environment / Notes" section if not relevant.
|
||||||
|
|
||||||
|
## Step 4 — Clarify
|
||||||
|
|
||||||
|
Show the draft to the user.
|
||||||
|
**Use `AskUserQuestion` tool to ask:**
|
||||||
|
- Are steps to reproduce complete and accurate?
|
||||||
|
- Severity: Blocker / Critical / Major / Minor / Trivial?
|
||||||
|
- Any related tickets or recent changes to link?
|
||||||
|
|
||||||
|
Incorporate feedback. Repeat until user approves.
|
||||||
|
|
||||||
|
## Step 5 — Determine Project
|
||||||
|
|
||||||
|
- Frontend / UI / UX → project: `NCWF`
|
||||||
|
- Backend / coordinator / systems / bot / engine → project: `NCS`
|
||||||
|
|
||||||
|
If ambiguous, ask the user.
|
||||||
|
|
||||||
|
## Step 6 — Create Issue
|
||||||
|
|
||||||
|
Call `mcp__youtrack__create_issue` with:
|
||||||
|
- `project`: determined in Step 5
|
||||||
|
- `summary`: concise title describing what is broken (≤72 chars, sentence case)
|
||||||
|
- `description`: full formatted defect report from Step 3 (Markdown)
|
||||||
|
- `type`: `Bug`
|
||||||
|
|
||||||
|
## Step 7 — Report
|
||||||
|
|
||||||
|
Display the created issue ID and URL.
|
||||||
|
Ask if a linked investigation or fix task is needed.
|
||||||
@@ -0,0 +1,112 @@
|
|||||||
|
# Create User Story in YouTrack
|
||||||
|
|
||||||
|
Automated user-story creation workflow. Topic/hint: `$ARGUMENTS`
|
||||||
|
|
||||||
|
## Step 1 — Gather Context
|
||||||
|
|
||||||
|
Use `AskUserQuestion` tool to ask the user (max 4 questions at once):
|
||||||
|
|
||||||
|
1. **Domain** — Is this frontend (UI/UX) or backend/coordinator/systems work?
|
||||||
|
2. **User type** — Who is the actor? (e.g. player, admin, bot, system)
|
||||||
|
3. **Action** — What should the user be able to do?
|
||||||
|
4. **Goal/value** — Why? What outcome does it enable?
|
||||||
|
|
||||||
|
If `$ARGUMENTS` already answers some of these, skip those questions.
|
||||||
|
|
||||||
|
## Step 2 — Research (if needed)
|
||||||
|
|
||||||
|
If the topic involves unfamiliar domain logic, game rules, or technical constraints:
|
||||||
|
- Search the repo for relevant code (use `Grep`/`Bash` to find related files).
|
||||||
|
- Use `WebSearch` if the topic involves external standards or protocols.
|
||||||
|
- Do NOT guess. Surface findings before drafting.
|
||||||
|
|
||||||
|
## Step 3 — Draft Story
|
||||||
|
|
||||||
|
Compose the full story using this template:
|
||||||
|
|
||||||
|
```
|
||||||
|
As a [type of user]
|
||||||
|
I want to [perform an action]
|
||||||
|
So that [achieve a goal or value]
|
||||||
|
|
||||||
|
|
||||||
|
Description
|
||||||
|
|
||||||
|
[Additional context or business logic for this story.]
|
||||||
|
|
||||||
|
|
||||||
|
Acceptance Criteria
|
||||||
|
|
||||||
|
[List the specific, measurable criteria that define when this story is done:]
|
||||||
|
|
||||||
|
- Criterion 1
|
||||||
|
- Criterion 2
|
||||||
|
- Criterion 3
|
||||||
|
|
||||||
|
|
||||||
|
Implementation Notes
|
||||||
|
|
||||||
|
[Technical notes, design references, or constraints.]
|
||||||
|
```
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- User story line: plain English, present tense, from user's perspective.
|
||||||
|
- Acceptance criteria: testable, unambiguous, one condition each.
|
||||||
|
- Implementation notes: optional — only include if there are known constraints, related tickets, or design refs.
|
||||||
|
|
||||||
|
## Step 4 — Clarify Acceptance Criteria
|
||||||
|
|
||||||
|
Show the draft to the user.
|
||||||
|
**Use `AskUserQuestion` tool to ask:**
|
||||||
|
- Are the acceptance criteria complete and correct?
|
||||||
|
- Any implementation constraints to add?
|
||||||
|
- Priority (if known)?
|
||||||
|
|
||||||
|
Incorporate feedback. Repeat until user approves.
|
||||||
|
|
||||||
|
## Step 5 — Determine Project
|
||||||
|
|
||||||
|
> **Project routing rules (always apply these):**
|
||||||
|
> - Backend code (game engine, bots, API, services, coordinator) → `NCS`
|
||||||
|
> - Frontend code (UI, UX, web app) → `NCWF`
|
||||||
|
> - Infrastructure (Kubernetes, pipelines, CI/CD, DB setup, cloud infra) → `NCI`
|
||||||
|
> - If ambiguous, ask the user.
|
||||||
|
|
||||||
|
- Frontend / UI / UX → project: `NCWF`
|
||||||
|
- Backend / coordinator / systems / bot / engine → project: `NCS`
|
||||||
|
- Kubernetes, pipelines, CI/CD, DB setup, infrastructure → project: `NCI`
|
||||||
|
|
||||||
|
If still ambiguous, ask the user.
|
||||||
|
|
||||||
|
## Step 6 — Create Issue
|
||||||
|
|
||||||
|
Call `mcp__youtrack__create_issue` with:
|
||||||
|
- `project`: determined in Step 5
|
||||||
|
- `summary`: concise title derived from the "I want to" clause (≤72 chars, sentence case)
|
||||||
|
- `description`: full formatted story from Step 3 (Markdown)
|
||||||
|
- `type`: `Feature` (or `Task` if purely technical with no user-facing value)
|
||||||
|
|
||||||
|
## Step 7 — Link Issues
|
||||||
|
|
||||||
|
After creation, **automatically** ask the user (use `AskUserQuestion` if interactive, otherwise infer from context):
|
||||||
|
|
||||||
|
> Are there related issues to link? (skip if none)
|
||||||
|
|
||||||
|
Collect any issue IDs the user mentions. For each, determine the correct relation and call `mcp__youtrack__link_issues`:
|
||||||
|
|
||||||
|
| Situation | Relation to use |
|
||||||
|
|-----------|----------------|
|
||||||
|
| This story must be done before another | `blocks` |
|
||||||
|
| Another story must be done before this | `is blocked by` |
|
||||||
|
| Stories share domain or are related | `relates to` |
|
||||||
|
| This is a child of an epic/story | `subtask of` |
|
||||||
|
| This is a parent grouping subtasks | `parent for` |
|
||||||
|
| This depends on another ticket's output | `depends on` |
|
||||||
|
|
||||||
|
If the user mentions an issue in the story description or implementation notes (e.g. "see NCS-42", "after NCS-12 is done"), auto-detect and suggest linking it — confirm before creating the link.
|
||||||
|
|
||||||
|
## Step 8 — Report
|
||||||
|
|
||||||
|
Display the created issue ID and URL.
|
||||||
|
List any links created (relation type + linked issue ID).
|
||||||
|
Ask if a linked sub-task or implementation ticket is needed.
|
||||||
@@ -0,0 +1,90 @@
|
|||||||
|
# Estimate Issue Time in YouTrack
|
||||||
|
|
||||||
|
Sprint planning time estimator. Issue ID or empty for full current sprint: `$ARGUMENTS`
|
||||||
|
|
||||||
|
## Step 1 — Determine Scope
|
||||||
|
|
||||||
|
**Single-issue mode** (`$ARGUMENTS` set):
|
||||||
|
- Call `mcp__youtrack__get_issue` on `$ARGUMENTS`.
|
||||||
|
- Proceed with that issue only.
|
||||||
|
|
||||||
|
**Sprint mode** (`$ARGUMENTS` empty):
|
||||||
|
- Call `mcp__youtrack__search_issues` with query `project: NCS Sprints: {current sprint} #Unresolved`.
|
||||||
|
- If query returns 0 results, use `AskUserQuestion` to ask for the sprint name, then retry with `project: NCS Sprints: {name}`.
|
||||||
|
- Collect all returned issues.
|
||||||
|
|
||||||
|
## Step 2 — Build Issue Tree
|
||||||
|
|
||||||
|
For each top-level issue from Step 1:
|
||||||
|
1. Fetch full details via `mcp__youtrack__get_issue`: summary, description, acceptance criteria, Type, existing `Zeitschätzung`, linked issues.
|
||||||
|
2. Identify subtasks from links with relation `subtask of` (i.e. issues where the fetched issue is the parent).
|
||||||
|
3. Recursively fetch subtasks until all leaves are known.
|
||||||
|
4. Group into tree: Epic → Story → Task/Subtask.
|
||||||
|
|
||||||
|
**Leaf node** = issue with no subtask children.
|
||||||
|
**Parent node** = issue that has at least one subtask child.
|
||||||
|
|
||||||
|
## Step 3 — Estimate Leaf Nodes
|
||||||
|
|
||||||
|
For each leaf node:
|
||||||
|
1. Read: summary, description, acceptance criteria, implementation notes.
|
||||||
|
2. If scope is unclear, search codebase (`Grep`/`Bash`) for related files to gauge complexity.
|
||||||
|
3. Assign estimate using this scale:
|
||||||
|
|
||||||
|
| Size | Criteria | Estimate |
|
||||||
|
|------|----------|----------|
|
||||||
|
| Trivial | Config change, rename, 1-file tweak | 30m |
|
||||||
|
| Small | 1–3 files, clear scope, no unknowns | 1h–2h |
|
||||||
|
| Medium | 3–6 files, some design needed | 3h–5h |
|
||||||
|
| Large | 6+ files, cross-module, non-trivial design | 1d–2d |
|
||||||
|
| XL | New subsystem, research spike, major refactor | 3d–5d |
|
||||||
|
|
||||||
|
4. Record: estimate + one-line reasoning.
|
||||||
|
5. Skip leaf if it already has `Zeitschätzung` set — note it as pre-estimated.
|
||||||
|
|
||||||
|
## Step 4 — Roll Up for Display
|
||||||
|
|
||||||
|
YouTrack auto-sums `Zeitschätzung` from subtasks up to parents — **do not write estimates to parent nodes**.
|
||||||
|
|
||||||
|
Compute display-only rolled-up totals:
|
||||||
|
- Parent total = sum of all descendant leaf estimates (including pre-estimated ones).
|
||||||
|
- Flag any branch where some leaves are missing estimates (partial roll-up).
|
||||||
|
|
||||||
|
## Step 5 — Show Summary + Confirm
|
||||||
|
|
||||||
|
Display full tree with estimates. Format:
|
||||||
|
|
||||||
|
```
|
||||||
|
Epic NCS-10: Castling overhaul [4h 30m] ← rolled up
|
||||||
|
Story NCS-11: Validate castling rights [2h 30m] ← rolled up
|
||||||
|
Task NCS-12: Fix RuleSet check 1h 30m ← leaf (new)
|
||||||
|
Task NCS-13: Add regression tests 1h ← leaf (new)
|
||||||
|
Story NCS-14: UI feedback for castling [2h] ← rolled up
|
||||||
|
Task NCS-15: Highlight invalid squares 2h ← leaf (pre-set, skipped)
|
||||||
|
```
|
||||||
|
|
||||||
|
Legend: `[X]` = display-only roll-up (not written). Plain = will be written to YouTrack.
|
||||||
|
|
||||||
|
If sprint mode, show grand total at bottom:
|
||||||
|
```
|
||||||
|
Sprint total: Xd Yh Zm (N issues, M leaves to update)
|
||||||
|
```
|
||||||
|
|
||||||
|
**Use `AskUserQuestion` tool:**
|
||||||
|
- Does the breakdown look right?
|
||||||
|
- Any estimates to adjust before writing to YouTrack?
|
||||||
|
|
||||||
|
Incorporate all feedback before proceeding.
|
||||||
|
|
||||||
|
## Step 6 — Write Estimates
|
||||||
|
|
||||||
|
On user approval, write estimates **only to leaf nodes** (bottom-up order):
|
||||||
|
- For each leaf with a new estimate, call `mcp__youtrack__update_issue` with field `Zeitschätzung` = approved estimate.
|
||||||
|
- YouTrack period format: `"30m"`, `"1h 30m"`, `"1d"`, `"2d 4h"`.
|
||||||
|
- Skip leaves already pre-estimated.
|
||||||
|
|
||||||
|
## Step 7 — Report
|
||||||
|
|
||||||
|
List all updated issues with set estimates.
|
||||||
|
Show final rolled-up totals per Epic/Story (read back from YouTrack via `mcp__youtrack__get_issue` if needed).
|
||||||
|
In sprint mode, show total sprint estimate.
|
||||||
@@ -0,0 +1,140 @@
|
|||||||
|
# Fix Defect from YouTrack
|
||||||
|
|
||||||
|
Automated defect-fix workflow. Ticket ID: `$ARGUMENTS`
|
||||||
|
|
||||||
|
## Step 1 — Fetch Ticket
|
||||||
|
|
||||||
|
Call `mcp__youtrack__get_issue` with ID `$ARGUMENTS`.
|
||||||
|
Extract and display: summary, description, steps to reproduce, Priority, Subsystem.
|
||||||
|
|
||||||
|
## Step 2 — Create Worktree
|
||||||
|
|
||||||
|
Derive branch name from ticket:
|
||||||
|
- `type` from YouTrack issue type: `bug` → `fix`, `feature`/`task` → `feat`, `refactor` → `refactor`, else `chore`
|
||||||
|
- `scope` from affected module/component (kebab-case, omit if unclear)
|
||||||
|
- `description` from ticket summary: lowercase, kebab-case, max 40 chars, drop articles
|
||||||
|
|
||||||
|
Branch format: `<type>/<ticket-id>-<description>`
|
||||||
|
Example: `fix/NOW-123-castling-validation-failure`
|
||||||
|
|
||||||
|
Call `EnterWorktree` with that branch name.
|
||||||
|
All subsequent file work happens inside this worktree.
|
||||||
|
|
||||||
|
## Step 3 — Identify Root Cause (read-only)
|
||||||
|
|
||||||
|
1. Run `./compile` — capture all errors and warnings.
|
||||||
|
2. Run `./test` — capture all failures.
|
||||||
|
3. Spawn `cavecrew-investigator` with: ticket description + compile/test output → locate root cause (files, line numbers, what's wrong).
|
||||||
|
4. **If anything is ambiguous (reproduction unclear, scope uncertain, conflicting signals), use `AskUserQuestion` tool to ask — max 4 questions at once.**
|
||||||
|
5. **Report findings to user. No file writes yet. Wait for acknowledgement before continuing.**
|
||||||
|
|
||||||
|
## Step 3b — Complexity Assessment + Subtasks
|
||||||
|
|
||||||
|
After root cause confirmed, assess scope:
|
||||||
|
|
||||||
|
**Simple** (1–2 files, single concern, < 1 hour estimated): proceed directly to Step 4.
|
||||||
|
|
||||||
|
**Complex** (3+ files, multiple concerns, or estimated > 1 hour): create subtasks before coding.
|
||||||
|
|
||||||
|
To create subtasks:
|
||||||
|
1. Break fix into discrete, independently-completable tasks (e.g. "Fix validation in RuleSet", "Add regression test for castling edge case", "Update FenParser to handle X").
|
||||||
|
2. For each subtask call `mcp__youtrack__create_issue` with:
|
||||||
|
- `project`: based on subtask content — do **not** inherit from parent. Backend code → `NCS`; frontend/UI → `NCWF`; Kubernetes/pipelines/CI-CD/DB setup/infrastructure → `NCI`. If ambiguous, ask user.
|
||||||
|
- `summary`: concise action-oriented title
|
||||||
|
- `type`: `Task`
|
||||||
|
- `description`: what to do and why
|
||||||
|
3. Call `mcp__youtrack__link_issues` to link each subtask to `$ARGUMENTS` with relation `subtask of`.
|
||||||
|
4. Check if the ticket description or comments mention other issue IDs. For each mentioned ID, suggest a link and confirm with user:
|
||||||
|
- Fix depends on another fix finishing first → `is blocked by`
|
||||||
|
- This fix blocks another ticket → `blocks`
|
||||||
|
- Logically related but independent → `relates to`
|
||||||
|
5. List created subtask IDs and any additional links to user.
|
||||||
|
|
||||||
|
Then proceed to Step 4, implementing subtasks in order.
|
||||||
|
|
||||||
|
## Step 4 — Fix
|
||||||
|
|
||||||
|
1. Implement fix (use `scala-implementer` agent for non-trivial changes; inline edits for small ones).
|
||||||
|
2. Run `./compile` — must be green.
|
||||||
|
3. Run `./test` — must be green.
|
||||||
|
4. Run `./gradlew spotlessScalaApply` — **blocking, foreground only**. Wait for completion before continuing.
|
||||||
|
5. Run `./lint` — **blocking, foreground only** (never `run_in_background`). Wait for exit code 0. Must be green.
|
||||||
|
- If lint fails, fix all issues and re-run until exit code 0.
|
||||||
|
- **Do NOT proceed to Step 5 until `./lint` has completed and returned exit code 0.**
|
||||||
|
If any step fails, iterate until all pass.
|
||||||
|
|
||||||
|
## Step 5 — Review
|
||||||
|
|
||||||
|
Spawn `cavecrew-reviewer` on the full diff.
|
||||||
|
Display findings grouped by severity.
|
||||||
|
|
||||||
|
## Step 5b — Apply Review Findings
|
||||||
|
|
||||||
|
If the review produced any findings (any severity):
|
||||||
|
1. Implement all agreed fixes.
|
||||||
|
2. Run `./compile` — must be green.
|
||||||
|
3. Run `./test` — must be green.
|
||||||
|
4. Run `./gradlew spotlessScalaApply` — **blocking, foreground only**. Wait for completion.
|
||||||
|
5. Run `./lint` — **blocking, foreground only**. Wait for exit code 0.
|
||||||
|
- If lint fails, fix all issues and re-run until exit code 0.
|
||||||
|
6. Re-spawn `cavecrew-reviewer` on the updated diff to confirm all findings are resolved.
|
||||||
|
|
||||||
|
Repeat until review is clean or user explicitly accepts remaining findings.
|
||||||
|
|
||||||
|
## Step 6 — Confirm + Push
|
||||||
|
|
||||||
|
Show summary: ticket, branch, files changed, review findings.
|
||||||
|
**Use `AskUserQuestion` tool to ask for explicit approval before pushing.** Include any open questions about commit message scope or body if unclear.
|
||||||
|
|
||||||
|
On approval, commit following Conventional Commits:
|
||||||
|
|
||||||
|
```
|
||||||
|
<type>(<scope>): <short description, imperative, ≤50 chars>
|
||||||
|
|
||||||
|
<optional body: what changed and why, wrap at 72 chars>
|
||||||
|
|
||||||
|
Closes $ARGUMENTS
|
||||||
|
https://knockoutwhist.youtrack.cloud/issue/$ARGUMENTS
|
||||||
|
```
|
||||||
|
|
||||||
|
- `type`: same as branch type (`fix`, `feat`, `refactor`, `chore`, etc.)
|
||||||
|
- `scope`: affected module (`core`, `rule`, `api`, `bot`, `io`)
|
||||||
|
- Subject: imperative mood, no period, lowercase
|
||||||
|
- Footer `Closes $ARGUMENTS` and ticket URL always present
|
||||||
|
|
||||||
|
Push branch to remote.
|
||||||
|
|
||||||
|
## Step 7 — Comment on Ticket
|
||||||
|
|
||||||
|
After successful push, call `mcp__youtrack__add_issue_comment` on `$ARGUMENTS` with:
|
||||||
|
|
||||||
|
```
|
||||||
|
Branch `<branch-name>` pushed.
|
||||||
|
|
||||||
|
<one-sentence summary of what was changed and why>
|
||||||
|
|
||||||
|
Files changed:
|
||||||
|
- <file1>
|
||||||
|
- <file2>
|
||||||
|
```
|
||||||
|
|
||||||
|
## Step 7b — Additional Links
|
||||||
|
|
||||||
|
After commenting, ask the user if `$ARGUMENTS` should be linked to any other issues not already linked:
|
||||||
|
|
||||||
|
| Situation | Relation |
|
||||||
|
|-----------|---------|
|
||||||
|
| This fix blocks another open ticket | `blocks` |
|
||||||
|
| Another ticket must ship first | `is blocked by` |
|
||||||
|
| Related defect or story | `relates to` |
|
||||||
|
| Duplicate of another defect | `duplicates` |
|
||||||
|
|
||||||
|
Scan the ticket description and comments for any issue IDs that were mentioned but not yet linked. Suggest those automatically.
|
||||||
|
|
||||||
|
Call `mcp__youtrack__link_issues` for each confirmed link.
|
||||||
|
|
||||||
|
## Step 8 — Cleanup
|
||||||
|
|
||||||
|
Call `ExitWorktree` with `discard_changes: true` to delete the worktree.
|
||||||
|
(Branch was pushed in step 6 — commits are safe on remote; `discard_changes: true` bypasses the local-ahead guard.)
|
||||||
|
Report: branch pushed, ticket commented, links created, worktree deleted, done.
|
||||||
@@ -0,0 +1,101 @@
|
|||||||
|
# Implement Feature from YouTrack
|
||||||
|
|
||||||
|
Automated feature-implementation workflow. Ticket ID: `$ARGUMENTS`
|
||||||
|
|
||||||
|
## Step 1 — Fetch Ticket
|
||||||
|
|
||||||
|
Call `mcp__youtrack__get_issue` with ID `$ARGUMENTS`.
|
||||||
|
Extract and display: summary, description, acceptance criteria, Priority, Subsystem.
|
||||||
|
|
||||||
|
## Step 2 — Create Worktree
|
||||||
|
|
||||||
|
Derive branch name from ticket:
|
||||||
|
- `type` from YouTrack issue type: `feature`/`task` → `feat`, `refactor` → `refactor`, `bug` → `fix`, else `chore`
|
||||||
|
- `scope` from affected module/component (kebab-case, omit if unclear)
|
||||||
|
- `description` from ticket summary: lowercase, kebab-case, max 40 chars, drop articles
|
||||||
|
|
||||||
|
Branch format: `<type>/<ticket-id>-<description>`
|
||||||
|
Example: `feat/NOW-456-add-bot-difficulty-slider`
|
||||||
|
|
||||||
|
Call `EnterWorktree` with that branch name.
|
||||||
|
All subsequent file work happens inside this worktree.
|
||||||
|
|
||||||
|
## Step 3 — Understand Requirements (read-only)
|
||||||
|
|
||||||
|
1. Run `./compile` — confirm baseline is green.
|
||||||
|
2. Run `./test` — confirm baseline is green.
|
||||||
|
3. Spawn `cavecrew-investigator` with: ticket description + acceptance criteria → locate affected files, relevant types/interfaces, entry points, integration touch-points.
|
||||||
|
4. **If anything is ambiguous (scope unclear, acceptance criteria missing, design decisions needed), use `AskUserQuestion` tool to ask — max 4 questions at once.**
|
||||||
|
5. **Report plan to user: what will be added/changed, which files, which modules. No file writes yet. Wait for acknowledgement before continuing.**
|
||||||
|
|
||||||
|
## Step 4 — Implement
|
||||||
|
|
||||||
|
1. Implement feature (use `scala-implementer` agent for non-trivial changes; inline edits for small ones).
|
||||||
|
2. Run `./compile` — must be green.
|
||||||
|
3. Run `./test` — must be green (add new tests for new behaviour; do not modify existing tests unless requirements changed).
|
||||||
|
4. Run `./gradlew spotlessScalaApply` — **blocking, foreground only**. Wait for completion before continuing.
|
||||||
|
5. Run `./lint` — **blocking, foreground only** (never `run_in_background`). Wait for exit code 0. Must be green.
|
||||||
|
- If lint fails, fix all issues and re-run until exit code 0.
|
||||||
|
- **Do NOT proceed to Step 5 until `./lint` has completed and returned exit code 0.**
|
||||||
|
If any step fails, iterate until all pass.
|
||||||
|
|
||||||
|
## Step 5 — Review
|
||||||
|
|
||||||
|
Spawn `cavecrew-reviewer` on the full diff.
|
||||||
|
Display findings grouped by severity.
|
||||||
|
|
||||||
|
## Step 5b — Apply Review Findings
|
||||||
|
|
||||||
|
If the review produced any findings (any severity):
|
||||||
|
1. Implement all agreed fixes.
|
||||||
|
2. Run `./compile` — must be green.
|
||||||
|
3. Run `./test` — must be green.
|
||||||
|
4. Run `./gradlew spotlessScalaApply` — **blocking, foreground only**. Wait for completion.
|
||||||
|
5. Run `./lint` — **blocking, foreground only**. Wait for exit code 0.
|
||||||
|
- If lint fails, fix all issues and re-run until exit code 0.
|
||||||
|
6. Re-spawn `cavecrew-reviewer` on the updated diff to confirm all findings are resolved.
|
||||||
|
|
||||||
|
Repeat until review is clean or user explicitly accepts remaining findings.
|
||||||
|
|
||||||
|
## Step 6 — Confirm + Push
|
||||||
|
|
||||||
|
Show summary: ticket, branch, files changed, review findings.
|
||||||
|
**Use `AskUserQuestion` tool to ask for explicit approval before pushing.** Include any open questions about commit message scope or body if unclear.
|
||||||
|
|
||||||
|
On approval, commit following Conventional Commits:
|
||||||
|
|
||||||
|
```
|
||||||
|
<type>(<scope>): <short description, imperative, ≤50 chars>
|
||||||
|
|
||||||
|
<optional body: what changed and why, wrap at 72 chars>
|
||||||
|
|
||||||
|
Closes $ARGUMENTS
|
||||||
|
https://knockoutwhist.youtrack.cloud/issue/$ARGUMENTS
|
||||||
|
```
|
||||||
|
|
||||||
|
- `type`: same as branch type (`feat`, `refactor`, `chore`, etc.)
|
||||||
|
- `scope`: affected module (`core`, `rule`, `api`, `bot`, `io`)
|
||||||
|
- Subject: imperative mood, no period, lowercase
|
||||||
|
- Footer `Closes $ARGUMENTS` and ticket URL always present
|
||||||
|
|
||||||
|
Push branch to remote.
|
||||||
|
|
||||||
|
## Step 7 — Comment on Ticket
|
||||||
|
|
||||||
|
After successful push, call `mcp__youtrack__add_issue_comment` on `$ARGUMENTS` with:
|
||||||
|
|
||||||
|
```
|
||||||
|
Branch `<branch-name>` pushed.
|
||||||
|
|
||||||
|
<one-sentence summary of what was added and why>
|
||||||
|
|
||||||
|
Files changed:
|
||||||
|
- <file1>
|
||||||
|
- <file2>
|
||||||
|
```
|
||||||
|
|
||||||
|
## Step 8 — Cleanup
|
||||||
|
|
||||||
|
Call `ExitWorktree` with `discard_changes: true` to delete the worktree.
|
||||||
|
(Branch was pushed in step 6 — commits are safe on remote; `discard_changes: true` bypasses the local-ahead guard.)
|
||||||
|
Report: branch pushed, ticket commented, worktree deleted, done.
|
||||||
@@ -0,0 +1,109 @@
|
|||||||
|
# Split Story into Subtasks in YouTrack
|
||||||
|
|
||||||
|
Split a user story into smaller, implementable subtasks. Story ID: `$ARGUMENTS`
|
||||||
|
|
||||||
|
## Step 1 — Fetch Story
|
||||||
|
|
||||||
|
Call `mcp__youtrack__get_issue` with ID `$ARGUMENTS`.
|
||||||
|
Extract and display: summary, description, acceptance criteria, implementation notes.
|
||||||
|
|
||||||
|
## Step 2 — Research (if needed)
|
||||||
|
|
||||||
|
If the story involves unfamiliar domain logic or technical constraints:
|
||||||
|
- Search repo for relevant code (`Grep`/`Bash`).
|
||||||
|
- Use `WebSearch` for external standards or protocols.
|
||||||
|
- Do NOT guess. Surface findings before proposing splits.
|
||||||
|
|
||||||
|
## Step 3 — Propose Split
|
||||||
|
|
||||||
|
Analyse the story and propose a set of subtasks. Rules:
|
||||||
|
- Each subtask = one unit of work, completable independently or in sequence.
|
||||||
|
- No subtask should exceed ~2 days of work.
|
||||||
|
- Name subtasks in imperative mood (e.g. "Implement move validation endpoint").
|
||||||
|
- Cover the full scope of the parent story — no gaps.
|
||||||
|
|
||||||
|
Show proposed subtask list to user (titles only) and ask:
|
||||||
|
**Use `AskUserQuestion` tool:**
|
||||||
|
- Does the split look right?
|
||||||
|
- Any subtasks to add, remove, or merge?
|
||||||
|
- Should any subtask be assigned to a specific person?
|
||||||
|
|
||||||
|
Incorporate feedback. Repeat until user approves the list.
|
||||||
|
|
||||||
|
## Step 4 — Draft Each Subtask
|
||||||
|
|
||||||
|
For each approved subtask, compose description using this template:
|
||||||
|
|
||||||
|
```
|
||||||
|
[Brief description of what needs to be done for this subtask.]
|
||||||
|
|
||||||
|
|
||||||
|
Steps / Tasks
|
||||||
|
|
||||||
|
- Task 1
|
||||||
|
- Task 2
|
||||||
|
- Task 3
|
||||||
|
|
||||||
|
|
||||||
|
Definition of Done
|
||||||
|
|
||||||
|
What must be true for this subtask to be considered complete:
|
||||||
|
|
||||||
|
- Code implemented
|
||||||
|
- Tests passed
|
||||||
|
- Reviewed and merged
|
||||||
|
```
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Steps/Tasks: concrete, ordered where order matters.
|
||||||
|
- Definition of Done: adjust per subtask — not all subtasks need the same criteria (e.g. a research spike has different DoD than an implementation task).
|
||||||
|
- Keep description short — one paragraph max.
|
||||||
|
|
||||||
|
## Step 5 — Determine Project per Subtask
|
||||||
|
|
||||||
|
Assign each subtask's project based on its content — do **not** inherit blindly from parent:
|
||||||
|
|
||||||
|
- Backend code (game engine, bots, API, services, coordinator) → `NCS`
|
||||||
|
- Frontend code (UI, UX, web app) → `NCWF`
|
||||||
|
- Kubernetes, pipelines, CI/CD, DB setup, infrastructure → `NCI`
|
||||||
|
|
||||||
|
If a subtask's project is ambiguous, ask the user before creating it.
|
||||||
|
|
||||||
|
## Step 6 — Create Subtasks
|
||||||
|
|
||||||
|
For each subtask call `mcp__youtrack__create_issue` with:
|
||||||
|
- `project`: from Step 5
|
||||||
|
- `summary`: subtask title (≤72 chars, sentence case)
|
||||||
|
- `description`: full formatted description from Step 4 (Markdown)
|
||||||
|
- `type`: `Task`
|
||||||
|
|
||||||
|
Then call `mcp__youtrack__link_issues` to link each created subtask to `$ARGUMENTS` with relation `subtask of`.
|
||||||
|
|
||||||
|
## Step 6b — Inter-Subtask Links
|
||||||
|
|
||||||
|
If subtasks must be done in sequence (one depends on output of another), add ordering links:
|
||||||
|
- For each dependency pair call `mcp__youtrack__link_issues` with relation `is blocked by` (subtask B is blocked by subtask A).
|
||||||
|
|
||||||
|
Ask the user to confirm sequencing before adding these links:
|
||||||
|
|
||||||
|
> Do any subtasks have ordering dependencies? (e.g. "Implement X must come before Add tests for X")
|
||||||
|
|
||||||
|
## Step 6c — External Links
|
||||||
|
|
||||||
|
Scan `$ARGUMENTS` description and implementation notes for any referenced issue IDs not already linked. For each:
|
||||||
|
|
||||||
|
| Situation | Relation |
|
||||||
|
|-----------|---------|
|
||||||
|
| Parent story blocks another epic/story | `blocks` |
|
||||||
|
| Story depends on another epic completing | `is blocked by` |
|
||||||
|
| Related story in same domain | `relates to` |
|
||||||
|
| This story duplicates or supersedes | `duplicates` |
|
||||||
|
|
||||||
|
Suggest links to the user and call `mcp__youtrack__link_issues` on confirmation.
|
||||||
|
|
||||||
|
## Step 7 — Report
|
||||||
|
|
||||||
|
List all created subtask IDs and summaries.
|
||||||
|
List all links created (subtask-of, blocking chains, external).
|
||||||
|
Display parent story link.
|
||||||
|
Ask if any subtask needs further splitting.
|
||||||
@@ -48,3 +48,4 @@ graphify-out/
|
|||||||
.DS_Store
|
.DS_Store
|
||||||
/jacoco-reporter/.venv/
|
/jacoco-reporter/.venv/
|
||||||
/.claude/settings.local.json
|
/.claude/settings.local.json
|
||||||
|
/.claude/worktrees/
|
||||||
|
|||||||
@@ -57,6 +57,14 @@ Use consistently.
|
|||||||
- **Tests are the spec.** Don't modify to pass. Fix requirements/code. Update only if requirements change.
|
- **Tests are the spec.** Don't modify to pass. Fix requirements/code. Update only if requirements change.
|
||||||
- Never read build folders. Ask permission if needed.
|
- Never read build folders. Ask permission if needed.
|
||||||
- Keep file current with decisions + conventions.
|
- Keep file current with decisions + conventions.
|
||||||
|
- **NativeReflectionConfig (mandatory):** Every new type (class, case class, enum, sealed trait — anything serialized) must be registered in the `NativeReflectionConfig` of **every module that interacts with it**. Configs live at:
|
||||||
|
- `modules/account/src/main/scala/de/nowchess/account/config/NativeReflectionConfig.scala`
|
||||||
|
- `modules/coordinator/src/main/scala/de/nowchess/coordinator/config/NativeReflectionConfig.scala`
|
||||||
|
- `modules/core/src/main/scala/de/nowchess/chess/config/NativeReflectionConfig.scala`
|
||||||
|
- `modules/io/src/main/scala/de/nowchess/io/service/config/NativeReflectionConfig.scala`
|
||||||
|
- `modules/rule/src/main/scala/de/nowchess/rules/config/NativeReflectionConfig.scala`
|
||||||
|
- `modules/store/src/main/scala/de/nowchess/store/config/NativeReflectionConfig.scala`
|
||||||
|
- `modules/ws/src/main/scala/de/nowchess/ws/config/NativeReflectionConfig.scala`
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
|||||||
@@ -348,3 +348,108 @@
|
|||||||
|
|
||||||
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-03)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* **account:** implement token pair handling for login and refresh endpoints ([9296db8](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/9296db88b7131bbda9b9b0da65c327ef9063ee31))
|
||||||
|
* add authentication permissions for metrics endpoints in application.yml ([04edd4d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/04edd4d6fd8a63196c36f6d67992832febc9bebb))
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **config:** add H2 database configuration for testing environment ([39c9e49](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/39c9e492cef2515368c074da9406f95e9c0c9e64))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* **docker:** add .dockerignore and .gitignore files for build exclusions ([c987d8e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c987d8e258c0e6c4cfbdaa8381c64c410d7a2b83))
|
||||||
|
* **docker:** add Dockerfiles for building Quarkus application in native and JVM modes ([3f2d2bb](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f2d2bb4c97fa8cddba66e1da4427c54236dfeed))
|
||||||
|
* **docker:** add Dockerfiles for Quarkus application in JVM and native modes ([34b9933](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/34b993304670cf2aa62cd2f6460cee7b9864b08e))
|
||||||
|
* **logging:** add DEBUG/INFO/WARN logging across services (NCS-72) ([#41](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/41)) ([804a4bf](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/804a4bf179e3dfb19e2be4390e7e543caf5237c6))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#46](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/46)) ([649566e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/649566eb3fcf38f91c8896a739f74ea318af312d))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#47](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/47)) ([87dfc6c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87dfc6c2bcce7f7d58fc641bd8d468a2e584c108))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
* update application.yml with new API root paths and add Micrometer and OpenTelemetry dependencies ([72ce262](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/72ce262bc491f94297700e6002fb5d0812e2cc2a))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* **account:** configure JDBC connection pool size to prevent exhaustion under load ([29072ef](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/29072efbfb1cfa1c3b1a85b4c1a587c971d245f9))
|
||||||
|
* **auth:** add InternalClientHeadersFactory for custom client headers management ([e279c39](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/e279c39246470156bf11e745ee72204018d4229d))
|
||||||
|
* NCS-84 More Verbose Logging ([#51](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/51)) ([4ad92ab](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/4ad92ab23698267f8faa59c4e18388d4a0042cca))
|
||||||
|
* **official-bots:** NCS-70-auto-register official bots with account service ([#59](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/59)) ([7117a93](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7117a93376272094d0b1a6abf2121254ce396684))
|
||||||
|
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
||||||
|
* **tests:** update token path to accessToken in ChallengeResourceTest ([354db11](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/354db11972342c47a1034303c11bccfb92e60109))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-05)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* **account:** implement token pair handling for login and refresh endpoints ([9296db8](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/9296db88b7131bbda9b9b0da65c327ef9063ee31))
|
||||||
|
* add authentication permissions for metrics endpoints in application.yml ([04edd4d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/04edd4d6fd8a63196c36f6d67992832febc9bebb))
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **api:** define shared EventEnvelope and EventType for Redis EventBus ([#61](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/61)) ([595c172](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/595c172900da99de367c274488c3ccbeaef55882))
|
||||||
|
* **config:** add H2 database configuration for testing environment ([39c9e49](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/39c9e492cef2515368c074da9406f95e9c0c9e64))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* **docker:** add .dockerignore and .gitignore files for build exclusions ([c987d8e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c987d8e258c0e6c4cfbdaa8381c64c410d7a2b83))
|
||||||
|
* **docker:** add Dockerfiles for building Quarkus application in native and JVM modes ([3f2d2bb](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f2d2bb4c97fa8cddba66e1da4427c54236dfeed))
|
||||||
|
* **docker:** add Dockerfiles for Quarkus application in JVM and native modes ([34b9933](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/34b993304670cf2aa62cd2f6460cee7b9864b08e))
|
||||||
|
* **logging:** add DEBUG/INFO/WARN logging across services (NCS-72) ([#41](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/41)) ([804a4bf](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/804a4bf179e3dfb19e2be4390e7e543caf5237c6))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#46](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/46)) ([649566e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/649566eb3fcf38f91c8896a739f74ea318af312d))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#47](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/47)) ([87dfc6c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87dfc6c2bcce7f7d58fc641bd8d468a2e584c108))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
* update application.yml with new API root paths and add Micrometer and OpenTelemetry dependencies ([72ce262](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/72ce262bc491f94297700e6002fb5d0812e2cc2a))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* **account:** configure JDBC connection pool size to prevent exhaustion under load ([29072ef](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/29072efbfb1cfa1c3b1a85b4c1a587c971d245f9))
|
||||||
|
* **auth:** add InternalClientHeadersFactory for custom client headers management ([e279c39](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/e279c39246470156bf11e745ee72204018d4229d))
|
||||||
|
* NCS-84 More Verbose Logging ([#51](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/51)) ([4ad92ab](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/4ad92ab23698267f8faa59c4e18388d4a0042cca))
|
||||||
|
* **official-bots:** NCS-70-auto-register official bots with account service ([#59](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/59)) ([7117a93](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7117a93376272094d0b1a6abf2121254ce396684))
|
||||||
|
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
||||||
|
* **tests:** update token path to accessToken in ChallengeResourceTest ([354db11](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/354db11972342c47a1034303c11bccfb92e60109))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-09)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* **account:** implement token pair handling for login and refresh endpoints ([9296db8](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/9296db88b7131bbda9b9b0da65c327ef9063ee31))
|
||||||
|
* add authentication permissions for metrics endpoints in application.yml ([04edd4d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/04edd4d6fd8a63196c36f6d67992832febc9bebb))
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **api:** define shared EventEnvelope and EventType for Redis EventBus ([#61](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/61)) ([595c172](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/595c172900da99de367c274488c3ccbeaef55882))
|
||||||
|
* **config:** add H2 database configuration for testing environment ([39c9e49](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/39c9e492cef2515368c074da9406f95e9c0c9e64))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* **docker:** add .dockerignore and .gitignore files for build exclusions ([c987d8e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c987d8e258c0e6c4cfbdaa8381c64c410d7a2b83))
|
||||||
|
* **docker:** add Dockerfiles for building Quarkus application in native and JVM modes ([3f2d2bb](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f2d2bb4c97fa8cddba66e1da4427c54236dfeed))
|
||||||
|
* **docker:** add Dockerfiles for Quarkus application in JVM and native modes ([34b9933](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/34b993304670cf2aa62cd2f6460cee7b9864b08e))
|
||||||
|
* **events:** migrate game-creation and bot flows to Redis Streams NCS-89 ([#62](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/62)) ([a24924c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a24924c23057db3d700a75dbc4333557789cd991))
|
||||||
|
* **logging:** add DEBUG/INFO/WARN logging across services (NCS-72) ([#41](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/41)) ([804a4bf](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/804a4bf179e3dfb19e2be4390e7e543caf5237c6))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#46](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/46)) ([649566e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/649566eb3fcf38f91c8896a739f74ea318af312d))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#47](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/47)) ([87dfc6c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87dfc6c2bcce7f7d58fc641bd8d468a2e584c108))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
* update application.yml with new API root paths and add Micrometer and OpenTelemetry dependencies ([72ce262](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/72ce262bc491f94297700e6002fb5d0812e2cc2a))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* **account:** configure JDBC connection pool size to prevent exhaustion under load ([29072ef](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/29072efbfb1cfa1c3b1a85b4c1a587c971d245f9))
|
||||||
|
* **auth:** add InternalClientHeadersFactory for custom client headers management ([e279c39](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/e279c39246470156bf11e745ee72204018d4229d))
|
||||||
|
* NCS-84 More Verbose Logging ([#51](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/51)) ([4ad92ab](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/4ad92ab23698267f8faa59c4e18388d4a0042cca))
|
||||||
|
* **official-bots:** NCS-70-auto-register official bots with account service ([#59](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/59)) ([7117a93](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7117a93376272094d0b1a6abf2121254ce396684))
|
||||||
|
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
||||||
|
* **tests:** update token path to accessToken in ChallengeResourceTest ([354db11](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/354db11972342c47a1034303c11bccfb92e60109))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
|||||||
@@ -45,6 +45,7 @@ dependencies {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
implementation(project(":modules:api"))
|
||||||
implementation(project(":modules:security"))
|
implementation(project(":modules:security"))
|
||||||
|
|
||||||
implementation(platform("${quarkusPlatformGroupId}:${quarkusPlatformArtifactId}:${quarkusPlatformVersion}"))
|
implementation(platform("${quarkusPlatformGroupId}:${quarkusPlatformArtifactId}:${quarkusPlatformVersion}"))
|
||||||
|
|||||||
+133
@@ -0,0 +1,133 @@
|
|||||||
|
package de.nowchess.account.client
|
||||||
|
|
||||||
|
import com.fasterxml.jackson.databind.ObjectMapper
|
||||||
|
import de.nowchess.account.config.RedisConfig
|
||||||
|
import de.nowchess.api.dto.{GameCreationRequestDto, GameCreationResponseDto, PlayerInfoDto, TimeControlDto}
|
||||||
|
import de.nowchess.api.game.GameMode
|
||||||
|
import de.nowchess.api.player.PlayerType
|
||||||
|
import de.nowchess.api.event.{EventEnvelope, EventType}
|
||||||
|
import io.quarkus.redis.datasource.RedisDataSource
|
||||||
|
import io.quarkus.redis.datasource.stream.{StreamMessage, XAddArgs, XGroupCreateArgs, XReadGroupArgs}
|
||||||
|
import io.quarkus.runtime.StartupEvent
|
||||||
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
|
import jakarta.enterprise.event.Observes
|
||||||
|
import jakarta.inject.Inject
|
||||||
|
import org.eclipse.microprofile.config.inject.ConfigProperty
|
||||||
|
import org.eclipse.microprofile.context.ManagedExecutor
|
||||||
|
import org.jboss.logging.Logger
|
||||||
|
import scala.compiletime.uninitialized
|
||||||
|
import scala.jdk.CollectionConverters.*
|
||||||
|
import scala.util.{Failure, Success, Try}
|
||||||
|
import java.time.Duration
|
||||||
|
import java.util.UUID
|
||||||
|
import java.util.concurrent.{CompletableFuture, ConcurrentHashMap, TimeUnit}
|
||||||
|
|
||||||
|
@ApplicationScoped
|
||||||
|
class GameCreationStreamClient:
|
||||||
|
|
||||||
|
// scalafix:off DisableSyntax.var
|
||||||
|
@Inject var redis: RedisDataSource = uninitialized
|
||||||
|
@Inject var redisConfig: RedisConfig = uninitialized
|
||||||
|
@Inject var objectMapper: ObjectMapper = uninitialized
|
||||||
|
@Inject var executor: ManagedExecutor = uninitialized
|
||||||
|
@ConfigProperty(name = "nowchess.game-creation-stream.enabled", defaultValue = "true")
|
||||||
|
private var streamEnabled: Boolean = true
|
||||||
|
// scalafix:on DisableSyntax.var
|
||||||
|
|
||||||
|
private val log = Logger.getLogger(classOf[GameCreationStreamClient])
|
||||||
|
private val instanceId = UUID.randomUUID().toString
|
||||||
|
private val groupName = s"account-game-creation-$instanceId"
|
||||||
|
private val consumerId = instanceId
|
||||||
|
private val maxStreamLen = 1000L
|
||||||
|
private val timeout = Duration.ofSeconds(10)
|
||||||
|
|
||||||
|
private val pending = new ConcurrentHashMap[String, CompletableFuture[GameCreationResponseDto]]()
|
||||||
|
|
||||||
|
private def requestStream: String = s"${redisConfig.prefix}:game-creation"
|
||||||
|
private def responseStream: String = s"${redisConfig.prefix}:game-creation-response"
|
||||||
|
|
||||||
|
def start(@Observes _ev: StartupEvent): Unit =
|
||||||
|
if streamEnabled then
|
||||||
|
createGroupIfAbsent()
|
||||||
|
executor.submit(
|
||||||
|
new Runnable:
|
||||||
|
def run(): Unit = pollLoop(),
|
||||||
|
)
|
||||||
|
log.infof("Game-creation response listener started (consumer=%s)", consumerId)
|
||||||
|
|
||||||
|
def createGame(req: CoreCreateGameRequest): GameCreationResponseDto =
|
||||||
|
val correlationId = UUID.randomUUID().toString
|
||||||
|
val future = new CompletableFuture[GameCreationResponseDto]()
|
||||||
|
pending.put(correlationId, future)
|
||||||
|
Try {
|
||||||
|
val payload = objectMapper.valueToTree[com.fasterxml.jackson.databind.JsonNode](toDto(req))
|
||||||
|
val envelope = EventEnvelope.of(EventType.GameCreationRequest, payload, Some(correlationId))
|
||||||
|
publish(requestStream, envelope)
|
||||||
|
future.get(timeout.toMillis, TimeUnit.MILLISECONDS)
|
||||||
|
} match
|
||||||
|
case Success(resp) =>
|
||||||
|
pending.remove(correlationId)
|
||||||
|
resp
|
||||||
|
case Failure(ex) =>
|
||||||
|
pending.remove(correlationId)
|
||||||
|
log.errorf(ex, "Game creation request %s failed", correlationId)
|
||||||
|
GameCreationResponseDto(None, Some("Game creation request timed out or failed"))
|
||||||
|
|
||||||
|
private def toDto(req: CoreCreateGameRequest): GameCreationRequestDto =
|
||||||
|
GameCreationRequestDto(
|
||||||
|
white = req.white.map(p => PlayerInfoDto(p.id, p.displayName, PlayerType.Human)),
|
||||||
|
black = req.black.map(p => PlayerInfoDto(p.id, p.displayName, PlayerType.Human)),
|
||||||
|
timeControl = req.timeControl.map(t => TimeControlDto(t.limitSeconds, t.incrementSeconds, t.daysPerMove)),
|
||||||
|
mode = req.mode.map(_ => GameMode.Authenticated),
|
||||||
|
)
|
||||||
|
|
||||||
|
private def createGroupIfAbsent(): Unit =
|
||||||
|
Try(
|
||||||
|
redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xgroupCreate(responseStream, groupName, "0", new XGroupCreateArgs().mkstream()),
|
||||||
|
) match
|
||||||
|
case Failure(ex) if Option(ex.getMessage).exists(_.contains("BUSYGROUP")) => ()
|
||||||
|
case Failure(ex) => log.warnf(ex, "Failed to create response consumer group")
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def pollLoop(): Unit =
|
||||||
|
while true do
|
||||||
|
Try {
|
||||||
|
val messages = redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xreadgroup(
|
||||||
|
groupName,
|
||||||
|
consumerId,
|
||||||
|
responseStream,
|
||||||
|
">",
|
||||||
|
new XReadGroupArgs().count(10).block(Duration.ofSeconds(2)),
|
||||||
|
)
|
||||||
|
Option(messages).foreach(_.forEach(handleResponse))
|
||||||
|
} match
|
||||||
|
case Failure(ex) => log.warnf(ex, "Error in game-creation response poll loop")
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def handleResponse(msg: StreamMessage[String, String, String]): Unit =
|
||||||
|
val json = msg.payload().get("data")
|
||||||
|
Try(objectMapper.readValue(json, classOf[EventEnvelope])) match
|
||||||
|
case Success(envelope) =>
|
||||||
|
envelope.correlationId.flatMap(id => Option(pending.remove(id))).foreach { future =>
|
||||||
|
Try(objectMapper.treeToValue(envelope.payload, classOf[GameCreationResponseDto])) match
|
||||||
|
case Success(resp) => future.complete(resp)
|
||||||
|
case Failure(ex) => future.completeExceptionally(ex)
|
||||||
|
}
|
||||||
|
case Failure(ex) => log.warnf(ex, "Unparseable game-creation response: %s", json)
|
||||||
|
ack(msg.id())
|
||||||
|
|
||||||
|
private def ack(id: String): Unit =
|
||||||
|
Try(redis.stream(classOf[String]).xack(responseStream, groupName, id)) match
|
||||||
|
case Failure(ex) => log.warnf(ex, "Failed to ack response %s", id)
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def publish(key: String, envelope: EventEnvelope): Unit =
|
||||||
|
val json = objectMapper.writeValueAsString(envelope)
|
||||||
|
redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xadd(key, new XAddArgs().maxlen(maxStreamLen).nearlyExactTrimming(), Map("data" -> json).asJava)
|
||||||
|
()
|
||||||
+13
@@ -12,10 +12,19 @@ import de.nowchess.account.domain.{
|
|||||||
UserAccount,
|
UserAccount,
|
||||||
}
|
}
|
||||||
import de.nowchess.account.dto.*
|
import de.nowchess.account.dto.*
|
||||||
|
import de.nowchess.api.dto.{
|
||||||
|
GameCreationRequestDto,
|
||||||
|
GameCreationResponseDto,
|
||||||
|
PlayerInfoDto as ApiPlayerInfoDto,
|
||||||
|
TimeControlDto as ApiTimeControlDto,
|
||||||
|
}
|
||||||
|
import de.nowchess.api.event.{EventEnvelope, EventType}
|
||||||
import io.quarkus.runtime.annotations.RegisterForReflection
|
import io.quarkus.runtime.annotations.RegisterForReflection
|
||||||
|
|
||||||
@RegisterForReflection(
|
@RegisterForReflection(
|
||||||
targets = Array(
|
targets = Array(
|
||||||
|
classOf[EventEnvelope],
|
||||||
|
classOf[EventType],
|
||||||
classOf[UserAccount],
|
classOf[UserAccount],
|
||||||
classOf[BotAccount],
|
classOf[BotAccount],
|
||||||
classOf[OfficialBotAccount],
|
classOf[OfficialBotAccount],
|
||||||
@@ -46,6 +55,10 @@ import io.quarkus.runtime.annotations.RegisterForReflection
|
|||||||
classOf[CoreCreateGameRequest],
|
classOf[CoreCreateGameRequest],
|
||||||
classOf[CoreGameResponse],
|
classOf[CoreGameResponse],
|
||||||
classOf[OfficialChallengeResponse],
|
classOf[OfficialChallengeResponse],
|
||||||
|
classOf[GameCreationRequestDto],
|
||||||
|
classOf[GameCreationResponseDto],
|
||||||
|
classOf[ApiPlayerInfoDto],
|
||||||
|
classOf[ApiTimeControlDto],
|
||||||
),
|
),
|
||||||
)
|
)
|
||||||
class NativeReflectionConfig
|
class NativeReflectionConfig
|
||||||
|
|||||||
@@ -49,3 +49,5 @@ case class RotatedTokenDto(token: String)
|
|||||||
case class OfficialBotAccountDto(id: String, name: String, rating: Int, createdAt: String)
|
case class OfficialBotAccountDto(id: String, name: String, rating: Int, createdAt: String)
|
||||||
|
|
||||||
case class OfficialChallengeResponse(gameId: String, botName: String, difficulty: Int)
|
case class OfficialChallengeResponse(gameId: String, botName: String, difficulty: Int)
|
||||||
|
|
||||||
|
case class SyncOfficialBotsRequest(bots: List[String])
|
||||||
|
|||||||
@@ -89,6 +89,13 @@ class OfficialBotAccountRepository:
|
|||||||
def findAll(): List[OfficialBotAccount] =
|
def findAll(): List[OfficialBotAccount] =
|
||||||
em.createQuery("FROM OfficialBotAccount", classOf[OfficialBotAccount]).getResultList.asScala.toList
|
em.createQuery("FROM OfficialBotAccount", classOf[OfficialBotAccount]).getResultList.asScala.toList
|
||||||
|
|
||||||
|
def findByName(name: String): Option[OfficialBotAccount] =
|
||||||
|
em.createQuery("FROM OfficialBotAccount WHERE name = :name", classOf[OfficialBotAccount])
|
||||||
|
.setParameter("name", name)
|
||||||
|
.getResultList
|
||||||
|
.asScala
|
||||||
|
.headOption
|
||||||
|
|
||||||
def persist(bot: OfficialBotAccount): OfficialBotAccount =
|
def persist(bot: OfficialBotAccount): OfficialBotAccount =
|
||||||
em.persist(bot)
|
em.persist(bot)
|
||||||
bot
|
bot
|
||||||
|
|||||||
@@ -4,6 +4,7 @@ import de.nowchess.account.domain.{BotAccount, OfficialBotAccount, UserAccount}
|
|||||||
import de.nowchess.account.dto.*
|
import de.nowchess.account.dto.*
|
||||||
import de.nowchess.account.error.AccountError
|
import de.nowchess.account.error.AccountError
|
||||||
import de.nowchess.account.service.AccountService
|
import de.nowchess.account.service.AccountService
|
||||||
|
import de.nowchess.security.InternalOnly
|
||||||
import jakarta.annotation.security.RolesAllowed
|
import jakarta.annotation.security.RolesAllowed
|
||||||
import jakarta.enterprise.context.ApplicationScoped
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
import jakarta.inject.Inject
|
import jakarta.inject.Inject
|
||||||
@@ -179,6 +180,13 @@ class AccountResource:
|
|||||||
createdAt = bot.createdAt.toString,
|
createdAt = bot.createdAt.toString,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
@POST
|
||||||
|
@Path("/official-bots/sync")
|
||||||
|
@InternalOnly
|
||||||
|
def syncOfficialBots(req: SyncOfficialBotsRequest): Response =
|
||||||
|
accountService.syncOfficialBots(req.bots)
|
||||||
|
Response.noContent().build()
|
||||||
|
|
||||||
@GET
|
@GET
|
||||||
@Path("/official-bots")
|
@Path("/official-bots")
|
||||||
def getOfficialBots: Response =
|
def getOfficialBots: Response =
|
||||||
|
|||||||
+3
-5
@@ -1,6 +1,6 @@
|
|||||||
package de.nowchess.account.resource
|
package de.nowchess.account.resource
|
||||||
|
|
||||||
import de.nowchess.account.client.{CoreCreateGameRequest, CoreGameClient, CorePlayerInfo}
|
import de.nowchess.account.client.{CoreCreateGameRequest, CorePlayerInfo, GameCreationStreamClient}
|
||||||
import de.nowchess.account.dto.{ErrorDto, OfficialChallengeResponse}
|
import de.nowchess.account.dto.{ErrorDto, OfficialChallengeResponse}
|
||||||
import de.nowchess.account.service.{AccountService, EventPublisher}
|
import de.nowchess.account.service.{AccountService, EventPublisher}
|
||||||
import jakarta.annotation.security.RolesAllowed
|
import jakarta.annotation.security.RolesAllowed
|
||||||
@@ -9,7 +9,6 @@ import jakarta.inject.Inject
|
|||||||
import jakarta.ws.rs.*
|
import jakarta.ws.rs.*
|
||||||
import jakarta.ws.rs.core.{MediaType, Response}
|
import jakarta.ws.rs.core.{MediaType, Response}
|
||||||
import org.eclipse.microprofile.jwt.JsonWebToken
|
import org.eclipse.microprofile.jwt.JsonWebToken
|
||||||
import org.eclipse.microprofile.rest.client.inject.RestClient
|
|
||||||
import org.jboss.logging.Logger
|
import org.jboss.logging.Logger
|
||||||
import scala.compiletime.uninitialized
|
import scala.compiletime.uninitialized
|
||||||
|
|
||||||
@@ -29,8 +28,7 @@ class OfficialChallengeResource:
|
|||||||
@Inject var botEventPublisher: EventPublisher = uninitialized
|
@Inject var botEventPublisher: EventPublisher = uninitialized
|
||||||
|
|
||||||
@Inject
|
@Inject
|
||||||
@RestClient
|
var gameCreationClient: GameCreationStreamClient = uninitialized
|
||||||
var coreGameClient: CoreGameClient = uninitialized
|
|
||||||
// scalafix:on
|
// scalafix:on
|
||||||
|
|
||||||
private val log = Logger.getLogger(classOf[OfficialChallengeResource])
|
private val log = Logger.getLogger(classOf[OfficialChallengeResource])
|
||||||
@@ -72,7 +70,7 @@ class OfficialChallengeResource:
|
|||||||
(CorePlayerInfo(bot.id.toString, bot.name), CorePlayerInfo(user.id.toString, user.username), "white")
|
(CorePlayerInfo(bot.id.toString, bot.name), CorePlayerInfo(user.id.toString, user.username), "white")
|
||||||
val req = CoreCreateGameRequest(Some(white), Some(black), None, Some("Authenticated"))
|
val req = CoreCreateGameRequest(Some(white), Some(black), None, Some("Authenticated"))
|
||||||
val gameId =
|
val gameId =
|
||||||
try Right(coreGameClient.createGame(req).gameId)
|
try gameCreationClient.createGame(req).gameId.toRight("Failed to create game")
|
||||||
catch case _ => Left("Failed to create game")
|
catch case _ => Left("Failed to create game")
|
||||||
gameId match
|
gameId match
|
||||||
case Left(err) =>
|
case Left(err) =>
|
||||||
|
|||||||
@@ -206,6 +206,17 @@ class AccountService:
|
|||||||
officialBotAccountRepository.persist(bot)
|
officialBotAccountRepository.persist(bot)
|
||||||
Right(bot)
|
Right(bot)
|
||||||
|
|
||||||
|
@Transactional
|
||||||
|
def syncOfficialBots(botNames: List[String]): Unit =
|
||||||
|
botNames.foreach { name =>
|
||||||
|
if officialBotAccountRepository.findByName(name).isEmpty then
|
||||||
|
val bot = new OfficialBotAccount()
|
||||||
|
bot.name = name
|
||||||
|
bot.createdAt = Instant.now()
|
||||||
|
officialBotAccountRepository.persist(bot)
|
||||||
|
log.infof("Auto-registered official bot: %s", name)
|
||||||
|
}
|
||||||
|
|
||||||
def getOfficialBotAccounts(): List[OfficialBotAccount] =
|
def getOfficialBotAccounts(): List[OfficialBotAccount] =
|
||||||
officialBotAccountRepository.findAll()
|
officialBotAccountRepository.findAll()
|
||||||
|
|
||||||
|
|||||||
@@ -1,12 +1,6 @@
|
|||||||
package de.nowchess.account.service
|
package de.nowchess.account.service
|
||||||
|
|
||||||
import de.nowchess.account.client.{
|
import de.nowchess.account.client.{CoreCreateGameRequest, CorePlayerInfo, CoreTimeControl, GameCreationStreamClient}
|
||||||
CoreCreateGameRequest,
|
|
||||||
CoreGameClient,
|
|
||||||
CoreGameResponse,
|
|
||||||
CorePlayerInfo,
|
|
||||||
CoreTimeControl,
|
|
||||||
}
|
|
||||||
import de.nowchess.account.domain.{Challenge, ChallengeColor, ChallengeStatus, DeclineReason}
|
import de.nowchess.account.domain.{Challenge, ChallengeColor, ChallengeStatus, DeclineReason}
|
||||||
import de.nowchess.account.dto.{
|
import de.nowchess.account.dto.{
|
||||||
ChallengeDto,
|
ChallengeDto,
|
||||||
@@ -23,7 +17,6 @@ import jakarta.annotation.PostConstruct
|
|||||||
import jakarta.enterprise.context.ApplicationScoped
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
import jakarta.inject.Inject
|
import jakarta.inject.Inject
|
||||||
import jakarta.transaction.Transactional
|
import jakarta.transaction.Transactional
|
||||||
import org.eclipse.microprofile.rest.client.inject.RestClient
|
|
||||||
import org.jboss.logging.Logger
|
import org.jboss.logging.Logger
|
||||||
import scala.compiletime.uninitialized
|
import scala.compiletime.uninitialized
|
||||||
|
|
||||||
@@ -45,8 +38,7 @@ class ChallengeService:
|
|||||||
var challengeRepository: ChallengeRepository = uninitialized
|
var challengeRepository: ChallengeRepository = uninitialized
|
||||||
|
|
||||||
@Inject
|
@Inject
|
||||||
@RestClient
|
var gameCreationClient: GameCreationStreamClient = uninitialized
|
||||||
var coreGameClient: CoreGameClient = uninitialized
|
|
||||||
|
|
||||||
@Inject
|
@Inject
|
||||||
var eventPublisher: EventPublisher = uninitialized
|
var eventPublisher: EventPublisher = uninitialized
|
||||||
@@ -187,7 +179,7 @@ class ChallengeService:
|
|||||||
val (white, black) = assignColors(challenge)
|
val (white, black) = assignColors(challenge)
|
||||||
val tc = buildTimeControl(challenge)
|
val tc = buildTimeControl(challenge)
|
||||||
val req = CoreCreateGameRequest(Some(white), Some(black), tc, Some("Authenticated"))
|
val req = CoreCreateGameRequest(Some(white), Some(black), tc, Some("Authenticated"))
|
||||||
Right(coreGameClient.createGame(req).gameId)
|
gameCreationClient.createGame(req).gameId.toRight(ChallengeError.GameCreationFailed)
|
||||||
catch case _ => Left(ChallengeError.GameCreationFailed)
|
catch case _ => Left(ChallengeError.GameCreationFailed)
|
||||||
|
|
||||||
private def assignColors(challenge: Challenge): (CorePlayerInfo, CorePlayerInfo) =
|
private def assignColors(challenge: Challenge): (CorePlayerInfo, CorePlayerInfo) =
|
||||||
|
|||||||
@@ -1,31 +1,61 @@
|
|||||||
package de.nowchess.account.service
|
package de.nowchess.account.service
|
||||||
|
|
||||||
|
import com.fasterxml.jackson.databind.ObjectMapper
|
||||||
import de.nowchess.account.config.RedisConfig
|
import de.nowchess.account.config.RedisConfig
|
||||||
|
import de.nowchess.api.event.{EventEnvelope, EventType}
|
||||||
import io.quarkus.redis.datasource.RedisDataSource
|
import io.quarkus.redis.datasource.RedisDataSource
|
||||||
|
import io.quarkus.redis.datasource.stream.XAddArgs
|
||||||
import jakarta.enterprise.context.ApplicationScoped
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
import jakarta.inject.Inject
|
import jakarta.inject.Inject
|
||||||
import scala.compiletime.uninitialized
|
import scala.compiletime.uninitialized
|
||||||
|
import scala.jdk.CollectionConverters.*
|
||||||
|
|
||||||
@ApplicationScoped
|
@ApplicationScoped
|
||||||
class EventPublisher:
|
class EventPublisher:
|
||||||
|
|
||||||
// scalafix:off DisableSyntax.var
|
// scalafix:off DisableSyntax.var
|
||||||
@Inject var redis: RedisDataSource = uninitialized
|
@Inject var redis: RedisDataSource = uninitialized
|
||||||
@Inject var redisConfig: RedisConfig = uninitialized
|
@Inject var redisConfig: RedisConfig = uninitialized
|
||||||
|
@Inject var objectMapper: ObjectMapper = uninitialized
|
||||||
// scalafix:on DisableSyntax.var
|
// scalafix:on DisableSyntax.var
|
||||||
|
|
||||||
|
private val maxStreamLen = 1000L
|
||||||
|
|
||||||
def publishGameStart(botId: String, gameId: String, playingAs: String, difficulty: Int, botAccountId: String): Unit =
|
def publishGameStart(botId: String, gameId: String, playingAs: String, difficulty: Int, botAccountId: String): Unit =
|
||||||
val event =
|
val payload = objectMapper.createObjectNode()
|
||||||
s"""{"type":"gameStart","gameId":"$gameId","playingAs":"$playingAs","difficulty":$difficulty,"botAccountId":"$botAccountId"}"""
|
payload.put("gameId", gameId)
|
||||||
redis.pubsub(classOf[String]).publish(s"${redisConfig.prefix}:bot:$botId:events", event)
|
payload.put("playingAs", playingAs)
|
||||||
|
payload.put("difficulty", difficulty)
|
||||||
|
payload.put("botAccountId", botAccountId)
|
||||||
|
val envelope = EventEnvelope.of(EventType.BotGameStart, payload)
|
||||||
|
val json = objectMapper.writeValueAsString(envelope)
|
||||||
|
redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xadd(
|
||||||
|
s"${redisConfig.prefix}:bot:$botId:events:stream",
|
||||||
|
new XAddArgs().maxlen(maxStreamLen).nearlyExactTrimming(),
|
||||||
|
Map("data" -> json).asJava,
|
||||||
|
)
|
||||||
|
redis.pubsub(classOf[String]).publish(s"${redisConfig.prefix}:bot:$botId:events", json)
|
||||||
()
|
()
|
||||||
|
|
||||||
def publishChallengeCreated(destUserId: String, challengeId: String, challengerName: String): Unit =
|
def publishChallengeCreated(destUserId: String, challengeId: String, challengerName: String): Unit =
|
||||||
val event = s"""{"type":"challengeCreated","challengeId":"$challengeId","challengerName":"$challengerName"}"""
|
val payload = objectMapper.createObjectNode()
|
||||||
redis.pubsub(classOf[String]).publish(s"${redisConfig.prefix}:user:$destUserId:events", event)
|
payload.put("challengeId", challengeId)
|
||||||
()
|
payload.put("challengerName", challengerName)
|
||||||
|
publish(s"${redisConfig.prefix}:user:$destUserId:events", EventType.ChallengeCreated, payload)
|
||||||
|
|
||||||
def publishChallengeAccepted(challengerId: String, challengeId: String, gameId: String): Unit =
|
def publishChallengeAccepted(challengerId: String, challengeId: String, gameId: String): Unit =
|
||||||
val event = s"""{"type":"challengeAccepted","challengeId":"$challengeId","gameId":"$gameId"}"""
|
val payload = objectMapper.createObjectNode()
|
||||||
redis.pubsub(classOf[String]).publish(s"${redisConfig.prefix}:user:$challengerId:events", event)
|
payload.put("challengeId", challengeId)
|
||||||
|
payload.put("gameId", gameId)
|
||||||
|
publish(s"${redisConfig.prefix}:user:$challengerId:events", EventType.ChallengeAccepted, payload)
|
||||||
|
|
||||||
|
private def publish(
|
||||||
|
channel: String,
|
||||||
|
eventType: EventType,
|
||||||
|
payload: com.fasterxml.jackson.databind.node.ObjectNode,
|
||||||
|
): Unit =
|
||||||
|
val envelope = EventEnvelope.of(eventType, payload)
|
||||||
|
redis.pubsub(classOf[String]).publish(channel, objectMapper.writeValueAsString(envelope))
|
||||||
()
|
()
|
||||||
|
|||||||
@@ -34,3 +34,5 @@ nowchess:
|
|||||||
secret: test-secret
|
secret: test-secret
|
||||||
auth:
|
auth:
|
||||||
enabled: false
|
enabled: false
|
||||||
|
game-creation-stream:
|
||||||
|
enabled: false
|
||||||
|
|||||||
@@ -154,3 +154,35 @@ class AccountResourceTest:
|
|||||||
.post("/api/account/refresh")
|
.post("/api/account/refresh")
|
||||||
.`then`()
|
.`then`()
|
||||||
.statusCode(401)
|
.statusCode(401)
|
||||||
|
|
||||||
|
@Test
|
||||||
|
def syncOfficialBotsCreatesNewBots(): Unit =
|
||||||
|
givenRequest()
|
||||||
|
.body("""{"bots":["sync-easy","sync-hard"]}""")
|
||||||
|
.when()
|
||||||
|
.post("/api/account/official-bots/sync")
|
||||||
|
.`then`()
|
||||||
|
.statusCode(204)
|
||||||
|
RestAssured
|
||||||
|
.`given`()
|
||||||
|
.when()
|
||||||
|
.get("/api/account/official-bots")
|
||||||
|
.`then`()
|
||||||
|
.statusCode(200)
|
||||||
|
.body("name", hasItems("sync-easy", "sync-hard"))
|
||||||
|
|
||||||
|
@Test
|
||||||
|
def syncOfficialBotsIsIdempotent(): Unit =
|
||||||
|
val body = """{"bots":["idempotent-bot"]}"""
|
||||||
|
givenRequest()
|
||||||
|
.body(body)
|
||||||
|
.when()
|
||||||
|
.post("/api/account/official-bots/sync")
|
||||||
|
.`then`()
|
||||||
|
.statusCode(204)
|
||||||
|
givenRequest()
|
||||||
|
.body(body)
|
||||||
|
.when()
|
||||||
|
.post("/api/account/official-bots/sync")
|
||||||
|
.`then`()
|
||||||
|
.statusCode(204)
|
||||||
|
|||||||
+6
-5
@@ -1,11 +1,11 @@
|
|||||||
package de.nowchess.account.resource
|
package de.nowchess.account.resource
|
||||||
|
|
||||||
import de.nowchess.account.client.{CoreGameClient, CoreGameResponse}
|
import de.nowchess.account.client.GameCreationStreamClient
|
||||||
|
import de.nowchess.api.dto.GameCreationResponseDto
|
||||||
import io.quarkus.test.InjectMock
|
import io.quarkus.test.InjectMock
|
||||||
import io.quarkus.test.junit.QuarkusTest
|
import io.quarkus.test.junit.QuarkusTest
|
||||||
import io.restassured.RestAssured
|
import io.restassured.RestAssured
|
||||||
import io.restassured.http.ContentType
|
import io.restassured.http.ContentType
|
||||||
import org.eclipse.microprofile.rest.client.inject.RestClient
|
|
||||||
import org.hamcrest.Matchers.*
|
import org.hamcrest.Matchers.*
|
||||||
import org.junit.jupiter.api.{BeforeEach, Test}
|
import org.junit.jupiter.api.{BeforeEach, Test}
|
||||||
import org.mockito.{ArgumentMatchers, Mockito}
|
import org.mockito.{ArgumentMatchers, Mockito}
|
||||||
@@ -14,14 +14,15 @@ import org.mockito.{ArgumentMatchers, Mockito}
|
|||||||
class ChallengeResourceTest:
|
class ChallengeResourceTest:
|
||||||
|
|
||||||
@InjectMock
|
@InjectMock
|
||||||
@RestClient
|
|
||||||
// scalafix:off DisableSyntax.var
|
// scalafix:off DisableSyntax.var
|
||||||
var coreGameClient: CoreGameClient = scala.compiletime.uninitialized
|
var gameCreationClient: GameCreationStreamClient = scala.compiletime.uninitialized
|
||||||
// scalafix:on
|
// scalafix:on
|
||||||
|
|
||||||
@BeforeEach
|
@BeforeEach
|
||||||
def setup(): Unit =
|
def setup(): Unit =
|
||||||
Mockito.when(coreGameClient.createGame(ArgumentMatchers.any())).thenReturn(CoreGameResponse("test-game-id"))
|
Mockito
|
||||||
|
.when(gameCreationClient.createGame(ArgumentMatchers.any()))
|
||||||
|
.thenReturn(GameCreationResponseDto(Some("test-game-id")))
|
||||||
|
|
||||||
private def givenRequest() = RestAssured.`given`().contentType(ContentType.JSON)
|
private def givenRequest() = RestAssured.`given`().contentType(ContentType.JSON)
|
||||||
|
|
||||||
|
|||||||
@@ -1,3 +1,3 @@
|
|||||||
MAJOR=0
|
MAJOR=0
|
||||||
MINOR=18
|
MINOR=21
|
||||||
PATCH=0
|
PATCH=0
|
||||||
|
|||||||
@@ -145,3 +145,42 @@
|
|||||||
|
|
||||||
* **dependencies:** correct Jackson databind dependency group ID ([008d72d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/008d72d826707c04205bac7de25170fae5fed861))
|
* **dependencies:** correct Jackson databind dependency group ID ([008d72d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/008d72d826707c04205bac7de25170fae5fed861))
|
||||||
* IO microservice ([#38](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/38)) ([a386f57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a386f57c21d34ead6cc6f92836c52b714597e289))
|
* IO microservice ([#38](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/38)) ([a386f57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a386f57c21d34ead6cc6f92836c52b714597e289))
|
||||||
|
## (2026-06-05)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* **api:** define shared EventEnvelope and EventType for Redis EventBus ([#61](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/61)) ([595c172](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/595c172900da99de367c274488c3ccbeaef55882))
|
||||||
|
* **dto:** update GameWritebackEventDto for JSON deserialization and remove unused mixin ([576e3fe](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/576e3fea9bf1082549ea53efd3288474c42be93d))
|
||||||
|
* NCS-13 Implement Threefold Repetition ([#31](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/31)) ([767d305](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/767d3051a76c266050b6335774d66e2db2273c16))
|
||||||
|
* NCS-14 implemented insufficient moves rule ([#30](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/30)) ([b0399a4](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b0399a4e489950083066c9538df9a84dcc7a4613))
|
||||||
|
* NCS-21 Write Scripts to automate certain tasks ([#15](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/15)) ([8051871](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/80518719d536a087d339fe02530825dc07f8b388))
|
||||||
|
* NCS-25 Add linters to keep quality up ([#27](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/27)) ([fd4e67d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/fd4e67d4f782a7e955822d90cb909d0a81676fb2))
|
||||||
|
* NCS-37 Quarkus integration ([#35](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/35)) ([f088c4e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f088c4e9ffcc498d3d1b6f01e8f50042d5830d55))
|
||||||
|
* NCS-41 Bot Platform ([#33](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/33)) ([8744bee](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8744bee2dd20966dae90a09c21a43d5b06f59e00))
|
||||||
|
* **rule:** Rules as a microservice ([#39](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/39)) ([093134d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/093134d36c6844ba02a36a28d5d044f09291cd1d))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* **dependencies:** correct Jackson databind dependency group ID ([008d72d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/008d72d826707c04205bac7de25170fae5fed861))
|
||||||
|
* IO microservice ([#38](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/38)) ([a386f57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a386f57c21d34ead6cc6f92836c52b714597e289))
|
||||||
|
## (2026-06-09)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* **api:** define shared EventEnvelope and EventType for Redis EventBus ([#61](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/61)) ([595c172](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/595c172900da99de367c274488c3ccbeaef55882))
|
||||||
|
* **dto:** update GameWritebackEventDto for JSON deserialization and remove unused mixin ([576e3fe](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/576e3fea9bf1082549ea53efd3288474c42be93d))
|
||||||
|
* **events:** migrate game-creation and bot flows to Redis Streams NCS-89 ([#62](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/62)) ([a24924c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a24924c23057db3d700a75dbc4333557789cd991))
|
||||||
|
* NCS-13 Implement Threefold Repetition ([#31](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/31)) ([767d305](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/767d3051a76c266050b6335774d66e2db2273c16))
|
||||||
|
* NCS-14 implemented insufficient moves rule ([#30](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/30)) ([b0399a4](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b0399a4e489950083066c9538df9a84dcc7a4613))
|
||||||
|
* NCS-21 Write Scripts to automate certain tasks ([#15](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/15)) ([8051871](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/80518719d536a087d339fe02530825dc07f8b388))
|
||||||
|
* NCS-25 Add linters to keep quality up ([#27](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/27)) ([fd4e67d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/fd4e67d4f782a7e955822d90cb909d0a81676fb2))
|
||||||
|
* NCS-37 Quarkus integration ([#35](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/35)) ([f088c4e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f088c4e9ffcc498d3d1b6f01e8f50042d5830d55))
|
||||||
|
* NCS-41 Bot Platform ([#33](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/33)) ([8744bee](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8744bee2dd20966dae90a09c21a43d5b06f59e00))
|
||||||
|
* **rule:** Rules as a microservice ([#39](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/39)) ([093134d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/093134d36c6844ba02a36a28d5d044f09291cd1d))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* **dependencies:** correct Jackson databind dependency group ID ([008d72d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/008d72d826707c04205bac7de25170fae5fed861))
|
||||||
|
* IO microservice ([#38](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/38)) ([a386f57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a386f57c21d34ead6cc6f92836c52b714597e289))
|
||||||
|
|||||||
@@ -50,6 +50,8 @@ dependencies {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
implementation("com.fasterxml.jackson.core:jackson-databind:${versions["JACKSON"]!!}")
|
implementation("com.fasterxml.jackson.core:jackson-databind:${versions["JACKSON"]!!}")
|
||||||
|
implementation("com.fasterxml.jackson.module:jackson-module-scala_3:${versions["JACKSON_SCALA"]!!}")
|
||||||
|
implementation("com.fasterxml.jackson.datatype:jackson-datatype-jsr310:${versions["JACKSON"]!!}")
|
||||||
|
|
||||||
testImplementation(platform("org.junit:junit-bom:5.13.4"))
|
testImplementation(platform("org.junit:junit-bom:5.13.4"))
|
||||||
testImplementation("org.junit.jupiter:junit-jupiter")
|
testImplementation("org.junit.jupiter:junit-jupiter")
|
||||||
|
|||||||
@@ -0,0 +1,10 @@
|
|||||||
|
package de.nowchess.api.dto
|
||||||
|
|
||||||
|
import de.nowchess.api.game.GameMode
|
||||||
|
|
||||||
|
final case class GameCreationRequestDto(
|
||||||
|
white: Option[PlayerInfoDto],
|
||||||
|
black: Option[PlayerInfoDto],
|
||||||
|
timeControl: Option[TimeControlDto],
|
||||||
|
mode: Option[GameMode] = None,
|
||||||
|
)
|
||||||
@@ -0,0 +1,6 @@
|
|||||||
|
package de.nowchess.api.dto
|
||||||
|
|
||||||
|
final case class GameCreationResponseDto(
|
||||||
|
gameId: Option[String],
|
||||||
|
error: Option[String] = None,
|
||||||
|
)
|
||||||
@@ -0,0 +1,27 @@
|
|||||||
|
package de.nowchess.api.event
|
||||||
|
|
||||||
|
import com.fasterxml.jackson.databind.JsonNode
|
||||||
|
import java.time.Instant
|
||||||
|
import java.util.UUID
|
||||||
|
|
||||||
|
final case class EventEnvelope(
|
||||||
|
eventId: UUID,
|
||||||
|
`type`: EventType,
|
||||||
|
payload: JsonNode,
|
||||||
|
timestamp: Instant,
|
||||||
|
correlationId: Option[String],
|
||||||
|
)
|
||||||
|
|
||||||
|
object EventEnvelope:
|
||||||
|
def of(
|
||||||
|
`type`: EventType,
|
||||||
|
payload: JsonNode,
|
||||||
|
correlationId: Option[String] = None,
|
||||||
|
): EventEnvelope =
|
||||||
|
EventEnvelope(
|
||||||
|
eventId = UUID.randomUUID(),
|
||||||
|
`type` = `type`,
|
||||||
|
payload = payload,
|
||||||
|
timestamp = Instant.now(),
|
||||||
|
correlationId = correlationId,
|
||||||
|
)
|
||||||
@@ -0,0 +1,4 @@
|
|||||||
|
package de.nowchess.api.event
|
||||||
|
|
||||||
|
enum EventType:
|
||||||
|
case GameStart, GameCreationRequest, GameCreationResponse, BotGameStart, ChallengeCreated, ChallengeAccepted
|
||||||
@@ -0,0 +1,50 @@
|
|||||||
|
package de.nowchess.api.event
|
||||||
|
|
||||||
|
import com.fasterxml.jackson.databind.ObjectMapper
|
||||||
|
import com.fasterxml.jackson.module.scala.DefaultScalaModule
|
||||||
|
import org.scalatest.funsuite.AnyFunSuite
|
||||||
|
import org.scalatest.matchers.should.Matchers
|
||||||
|
|
||||||
|
class EventEnvelopeTest extends AnyFunSuite with Matchers:
|
||||||
|
|
||||||
|
private val mapper =
|
||||||
|
val m = new ObjectMapper()
|
||||||
|
m.registerModule(DefaultScalaModule)
|
||||||
|
m.findAndRegisterModules()
|
||||||
|
m
|
||||||
|
|
||||||
|
test("EventEnvelope round-trips through JSON") {
|
||||||
|
val payload = mapper.createObjectNode()
|
||||||
|
payload.put("gameId", "game-123")
|
||||||
|
payload.put("difficulty", 3)
|
||||||
|
|
||||||
|
val original = EventEnvelope.of(EventType.GameStart, payload, Some("corr-abc"))
|
||||||
|
|
||||||
|
val json = mapper.writeValueAsString(original)
|
||||||
|
val decoded = mapper.readValue(json, classOf[EventEnvelope])
|
||||||
|
|
||||||
|
decoded.eventId shouldBe original.eventId
|
||||||
|
decoded.`type` shouldBe original.`type`
|
||||||
|
decoded.payload shouldBe original.payload
|
||||||
|
decoded.timestamp shouldBe original.timestamp
|
||||||
|
decoded.correlationId shouldBe Some("corr-abc")
|
||||||
|
}
|
||||||
|
|
||||||
|
test("EventEnvelope serializes without correlationId") {
|
||||||
|
val payload = mapper.createObjectNode()
|
||||||
|
payload.put("challengeId", "ch-1")
|
||||||
|
|
||||||
|
val envelope = EventEnvelope.of(EventType.ChallengeCreated, payload)
|
||||||
|
val json = mapper.writeValueAsString(envelope)
|
||||||
|
val decoded = mapper.readValue(json, classOf[EventEnvelope])
|
||||||
|
|
||||||
|
decoded.`type` shouldBe EventType.ChallengeCreated
|
||||||
|
decoded.correlationId shouldBe None
|
||||||
|
}
|
||||||
|
|
||||||
|
test("EventEnvelope.of generates unique eventIds") {
|
||||||
|
val payload = mapper.createObjectNode()
|
||||||
|
val e1 = EventEnvelope.of(EventType.BotGameStart, payload)
|
||||||
|
val e2 = EventEnvelope.of(EventType.BotGameStart, payload)
|
||||||
|
e1.eventId should not equal e2.eventId
|
||||||
|
}
|
||||||
@@ -1,3 +1,3 @@
|
|||||||
MAJOR=0
|
MAJOR=0
|
||||||
MINOR=14
|
MINOR=16
|
||||||
PATCH=0
|
PATCH=0
|
||||||
|
|||||||
@@ -1759,3 +1759,140 @@
|
|||||||
|
|
||||||
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-02)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* add authentication permissions for metrics endpoints in application.yml ([04edd4d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/04edd4d6fd8a63196c36f6d67992832febc9bebb))
|
||||||
|
* add CORS configuration and reorder JWT settings in application.yml ([a49f9be](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a49f9be146f04c14561c305d980846a92f8c12b2))
|
||||||
|
* add GameRules stub with PositionStatus enum ([76d4168](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/76d4168038de23e5d6083d4e8f0504fbf31d15a3))
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add MovedInCheck/Checkmate/Stalemate MoveResult variants (stub dispatch) ([8b7ec57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8b7ec57e5ea6ee1615a1883848a426dc07d26364))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **config:** add GameWritebackEventDto to reflection targets ([87f29a7](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87f29a720422f538ef70699533500e060337b8ea))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* implement clock expiry scanning and handling for game records ([#53](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/53)) ([8f9eb12](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8f9eb12f663efabe4dc72b94394438652ad0ef02))
|
||||||
|
* implement GameRules with isInCheck, legalMoves, gameStatus ([94a02ff](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/94a02ff6849436d9496c70a0f16c21666dae8e4e))
|
||||||
|
* implement legal castling ([#1](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/1)) ([00d326c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/00d326c1ba67711fbe180f04e1100c3f01dd0254))
|
||||||
|
* implement periodic scaling checks and enhance instance management in AutoScaler ([3f12f69](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f12f695f132b92f634d98df2c037292498b6e86))
|
||||||
|
* **logging:** add DEBUG/INFO/WARN logging across services (NCS-72) ([#41](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/41)) ([804a4bf](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/804a4bf179e3dfb19e2be4390e7e543caf5237c6))
|
||||||
|
* NCS-10 Implement Pawn Promotion ([#12](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/12)) ([13bfc16](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/13bfc16cfe25db78ec607db523ca6d993c13430c))
|
||||||
|
* NCS-11 50-move rule ([#9](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/9)) ([412ed98](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/412ed986a95703a3b282276540153480ceed229d))
|
||||||
|
* NCS-13 Implement Threefold Repetition ([#31](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/31)) ([767d305](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/767d3051a76c266050b6335774d66e2db2273c16))
|
||||||
|
* NCS-14 implemented insufficient moves rule ([#30](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/30)) ([b0399a4](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b0399a4e489950083066c9538df9a84dcc7a4613))
|
||||||
|
* NCS-16 Core Separation via Patterns ([#10](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/10)) ([1361dfc](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/1361dfc89553b146864fb8ff3526cf12cf3f293a))
|
||||||
|
* NCS-17 Implement basic ScalaFX UI ([#14](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/14)) ([3ff8031](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ff80318b4f16c59733a46498581a5c27f048287))
|
||||||
|
* NCS-21 Write Scripts to automate certain tasks ([#15](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/15)) ([8051871](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/80518719d536a087d339fe02530825dc07f8b388))
|
||||||
|
* NCS-25 Add linters to keep quality up ([#27](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/27)) ([fd4e67d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/fd4e67d4f782a7e955822d90cb909d0a81676fb2))
|
||||||
|
* NCS-37 Quarkus integration ([#35](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/35)) ([f088c4e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f088c4e9ffcc498d3d1b6f01e8f50042d5830d55))
|
||||||
|
* NCS-40 Rework Draw System ([#34](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/34)) ([33e785d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/33e785d22af87724839b62ae91dfe74a05b398c3))
|
||||||
|
* NCS-41 Bot Platform ([#33](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/33)) ([8744bee](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8744bee2dd20966dae90a09c21a43d5b06f59e00))
|
||||||
|
* NCS-53 changed IO to MicroService for easier scaling ([#37](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/37)) ([b5a2966](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b5a2966adafa9650f0f7d601bdeb8fdd13710327))
|
||||||
|
* NCS-6 Implementing FEN & PGN ([#7](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/7)) ([f28e69d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f28e69dc181416aa2f221fdc4b45c2cda5efbf07))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#48](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/48)) ([c96a09b](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c96a09bb5cee59fc23205bb63baa8b217a7e1b00))
|
||||||
|
* NCS-9 En passant implementation ([#8](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/8)) ([919beb3](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/919beb3b4bfa8caf2f90976a415fe9b19b7e9747))
|
||||||
|
* **redis:** implement game writeback stream processing with error handling and retries ([ae3ef76](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/ae3ef766e8b7596a09e466cd4fb386119f17ca5c))
|
||||||
|
* **rule:** Rules as a microservice ([#39](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/39)) ([093134d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/093134d36c6844ba02a36a28d5d044f09291cd1d))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
* update application.yml with new API root paths and add Micrometer and OpenTelemetry dependencies ([72ce262](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/72ce262bc491f94297700e6002fb5d0812e2cc2a))
|
||||||
|
* wire check/checkmate/stalemate into processMove and gameLoop ([5264a22](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5264a225418b885c5e6ea6411b96f85e38837f6c))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* add missing kings to gameLoop capture test board ([aedd787](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/aedd787b77203c2af934751dba7b784eaf165032))
|
||||||
|
* **auth:** change InternalAuthFilter to use @Singleton and add HTTP tests for secret validation ([c08d530](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c08d5303eb9e70d36c8eebf6a061ccb71e118fe5))
|
||||||
|
* **auth:** update InternalAuthFilter to use @ApplicationScoped and add index-dependency configuration ([6e0fd95](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6e0fd9523e001756ce7109e639ebb54be4fcdabf))
|
||||||
|
* **core:** add logs to trace subscribeGame call in createGame ([f5614c3](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f5614c358255598ba1230e42a56b22934d79183c))
|
||||||
|
* correct test board positions and captureOutput/withInput interaction ([f0481e2](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f0481e2561b779df00925b46ee281dc36a795150))
|
||||||
|
* **heartbeat:** inject ObjectMapper into InstanceHeartbeatService ([#42](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/42)) ([0c98151](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/0c981517da1f94cd10ae396e47bde2b35d0b3ba0))
|
||||||
|
* IO microservice ([#38](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/38)) ([a386f57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a386f57c21d34ead6cc6f92836c52b714597e289))
|
||||||
|
* Lints ([dc224ab](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/dc224abe26acf5361c56956006e1cc51b75b0b7e))
|
||||||
|
* NCS-84 More Verbose Logging ([#51](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/51)) ([4ad92ab](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/4ad92ab23698267f8faa59c4e18388d4a0042cca))
|
||||||
|
* NCS-85 Database Writeback fails without Logs ([#52](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/52)) ([7323908](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/73239088d985f01aa6b1067ed9097a845e471d4f))
|
||||||
|
* **pgn:** add SAN disambiguation and check/checkmate suffixes [NCS-42] ([#56](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/56)) ([2579539](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2579539084152178f4482ddb7b84b7f1162f10da))
|
||||||
|
* **redis:** add max pool wait time and switch to ReactiveRedisDataSource for heartbeat updates ([33e5017](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/33e5017f51a998327b180f778f73964cc10c05d3))
|
||||||
|
* **redis:** enhance GameRedisSubscriberManager to use ReactiveRedisDataSource and improve subscription handling ([0eb752d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/0eb752d4935377f75aab710b7f4eda4b29098e6a))
|
||||||
|
* **redis:** prevent concurrent Redis heartbeat refreshes using AtomicBoolean ([847b132](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/847b13202cb909d18ca3304c27ebe17ce2312b8e))
|
||||||
|
* **redis:** simplify refreshRedisHeartbeat logic and ensure proper error handling ([1813ea1](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/1813ea1d2d5d093f7925f87371b5e29820bf1136))
|
||||||
|
* **redis:** update Redis configuration with max pool size and waiting parameters ([5baf6a7](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5baf6a7cdbea484fc49c02e2b5a1c3919b7fa2c4))
|
||||||
|
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
||||||
|
* resolve 6 coordinator bugs (cache eviction, rebalance race, pod matching, lookup inefficiency) ([5619c82](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5619c8223ad7091706909eda8c907a29d215fd30))
|
||||||
|
* update documentation to reflect new functions in CoordinatorGrpcServer and InstanceRegistry ([f7ce4df](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f7ce4df595cbdc2ef84122781f4851ff140c0f44))
|
||||||
|
* update main class path in build configuration and adjust VCS directory mapping ([7b1f8b1](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7b1f8b117623d327232a1a92a8a44d18582e0189))
|
||||||
|
* update move validation to check for king safety ([#13](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/13)) ([e5e20c5](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/e5e20c566e368b12ca1dc59680c34e9112bf6762))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-09)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* add authentication permissions for metrics endpoints in application.yml ([04edd4d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/04edd4d6fd8a63196c36f6d67992832febc9bebb))
|
||||||
|
* add CORS configuration and reorder JWT settings in application.yml ([a49f9be](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a49f9be146f04c14561c305d980846a92f8c12b2))
|
||||||
|
* add GameRules stub with PositionStatus enum ([76d4168](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/76d4168038de23e5d6083d4e8f0504fbf31d15a3))
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add MovedInCheck/Checkmate/Stalemate MoveResult variants (stub dispatch) ([8b7ec57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8b7ec57e5ea6ee1615a1883848a426dc07d26364))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **config:** add GameWritebackEventDto to reflection targets ([87f29a7](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87f29a720422f538ef70699533500e060337b8ea))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* **events:** migrate game-creation and bot flows to Redis Streams NCS-89 ([#62](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/62)) ([a24924c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a24924c23057db3d700a75dbc4333557789cd991))
|
||||||
|
* implement clock expiry scanning and handling for game records ([#53](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/53)) ([8f9eb12](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8f9eb12f663efabe4dc72b94394438652ad0ef02))
|
||||||
|
* implement GameRules with isInCheck, legalMoves, gameStatus ([94a02ff](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/94a02ff6849436d9496c70a0f16c21666dae8e4e))
|
||||||
|
* implement legal castling ([#1](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/1)) ([00d326c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/00d326c1ba67711fbe180f04e1100c3f01dd0254))
|
||||||
|
* implement periodic scaling checks and enhance instance management in AutoScaler ([3f12f69](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f12f695f132b92f634d98df2c037292498b6e86))
|
||||||
|
* **logging:** add DEBUG/INFO/WARN logging across services (NCS-72) ([#41](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/41)) ([804a4bf](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/804a4bf179e3dfb19e2be4390e7e543caf5237c6))
|
||||||
|
* NCS-10 Implement Pawn Promotion ([#12](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/12)) ([13bfc16](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/13bfc16cfe25db78ec607db523ca6d993c13430c))
|
||||||
|
* NCS-11 50-move rule ([#9](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/9)) ([412ed98](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/412ed986a95703a3b282276540153480ceed229d))
|
||||||
|
* NCS-13 Implement Threefold Repetition ([#31](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/31)) ([767d305](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/767d3051a76c266050b6335774d66e2db2273c16))
|
||||||
|
* NCS-14 implemented insufficient moves rule ([#30](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/30)) ([b0399a4](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b0399a4e489950083066c9538df9a84dcc7a4613))
|
||||||
|
* NCS-16 Core Separation via Patterns ([#10](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/10)) ([1361dfc](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/1361dfc89553b146864fb8ff3526cf12cf3f293a))
|
||||||
|
* NCS-17 Implement basic ScalaFX UI ([#14](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/14)) ([3ff8031](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ff80318b4f16c59733a46498581a5c27f048287))
|
||||||
|
* NCS-21 Write Scripts to automate certain tasks ([#15](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/15)) ([8051871](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/80518719d536a087d339fe02530825dc07f8b388))
|
||||||
|
* NCS-25 Add linters to keep quality up ([#27](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/27)) ([fd4e67d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/fd4e67d4f782a7e955822d90cb909d0a81676fb2))
|
||||||
|
* NCS-37 Quarkus integration ([#35](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/35)) ([f088c4e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f088c4e9ffcc498d3d1b6f01e8f50042d5830d55))
|
||||||
|
* NCS-40 Rework Draw System ([#34](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/34)) ([33e785d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/33e785d22af87724839b62ae91dfe74a05b398c3))
|
||||||
|
* NCS-41 Bot Platform ([#33](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/33)) ([8744bee](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8744bee2dd20966dae90a09c21a43d5b06f59e00))
|
||||||
|
* NCS-53 changed IO to MicroService for easier scaling ([#37](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/37)) ([b5a2966](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b5a2966adafa9650f0f7d601bdeb8fdd13710327))
|
||||||
|
* NCS-6 Implementing FEN & PGN ([#7](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/7)) ([f28e69d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f28e69dc181416aa2f221fdc4b45c2cda5efbf07))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#48](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/48)) ([c96a09b](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c96a09bb5cee59fc23205bb63baa8b217a7e1b00))
|
||||||
|
* NCS-9 En passant implementation ([#8](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/8)) ([919beb3](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/919beb3b4bfa8caf2f90976a415fe9b19b7e9747))
|
||||||
|
* **redis:** implement game writeback stream processing with error handling and retries ([ae3ef76](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/ae3ef766e8b7596a09e466cd4fb386119f17ca5c))
|
||||||
|
* **rule:** Rules as a microservice ([#39](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/39)) ([093134d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/093134d36c6844ba02a36a28d5d044f09291cd1d))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
* update application.yml with new API root paths and add Micrometer and OpenTelemetry dependencies ([72ce262](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/72ce262bc491f94297700e6002fb5d0812e2cc2a))
|
||||||
|
* wire check/checkmate/stalemate into processMove and gameLoop ([5264a22](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5264a225418b885c5e6ea6411b96f85e38837f6c))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* add missing kings to gameLoop capture test board ([aedd787](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/aedd787b77203c2af934751dba7b784eaf165032))
|
||||||
|
* **auth:** change InternalAuthFilter to use @Singleton and add HTTP tests for secret validation ([c08d530](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c08d5303eb9e70d36c8eebf6a061ccb71e118fe5))
|
||||||
|
* **auth:** update InternalAuthFilter to use @ApplicationScoped and add index-dependency configuration ([6e0fd95](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6e0fd9523e001756ce7109e639ebb54be4fcdabf))
|
||||||
|
* **core:** add logs to trace subscribeGame call in createGame ([f5614c3](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f5614c358255598ba1230e42a56b22934d79183c))
|
||||||
|
* correct test board positions and captureOutput/withInput interaction ([f0481e2](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f0481e2561b779df00925b46ee281dc36a795150))
|
||||||
|
* **heartbeat:** inject ObjectMapper into InstanceHeartbeatService ([#42](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/42)) ([0c98151](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/0c981517da1f94cd10ae396e47bde2b35d0b3ba0))
|
||||||
|
* IO microservice ([#38](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/38)) ([a386f57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a386f57c21d34ead6cc6f92836c52b714597e289))
|
||||||
|
* Lints ([dc224ab](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/dc224abe26acf5361c56956006e1cc51b75b0b7e))
|
||||||
|
* NCS-84 More Verbose Logging ([#51](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/51)) ([4ad92ab](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/4ad92ab23698267f8faa59c4e18388d4a0042cca))
|
||||||
|
* NCS-85 Database Writeback fails without Logs ([#52](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/52)) ([7323908](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/73239088d985f01aa6b1067ed9097a845e471d4f))
|
||||||
|
* **pgn:** add SAN disambiguation and check/checkmate suffixes [NCS-42] ([#56](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/56)) ([2579539](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2579539084152178f4482ddb7b84b7f1162f10da))
|
||||||
|
* **redis:** add max pool wait time and switch to ReactiveRedisDataSource for heartbeat updates ([33e5017](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/33e5017f51a998327b180f778f73964cc10c05d3))
|
||||||
|
* **redis:** enhance GameRedisSubscriberManager to use ReactiveRedisDataSource and improve subscription handling ([0eb752d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/0eb752d4935377f75aab710b7f4eda4b29098e6a))
|
||||||
|
* **redis:** prevent concurrent Redis heartbeat refreshes using AtomicBoolean ([847b132](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/847b13202cb909d18ca3304c27ebe17ce2312b8e))
|
||||||
|
* **redis:** simplify refreshRedisHeartbeat logic and ensure proper error handling ([1813ea1](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/1813ea1d2d5d093f7925f87371b5e29820bf1136))
|
||||||
|
* **redis:** update Redis configuration with max pool size and waiting parameters ([5baf6a7](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5baf6a7cdbea484fc49c02e2b5a1c3919b7fa2c4))
|
||||||
|
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
||||||
|
* resolve 6 coordinator bugs (cache eviction, rebalance race, pod matching, lookup inefficiency) ([5619c82](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5619c8223ad7091706909eda8c907a29d215fd30))
|
||||||
|
* update documentation to reflect new functions in CoordinatorGrpcServer and InstanceRegistry ([f7ce4df](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f7ce4df595cbdc2ef84122781f4851ff140c0f44))
|
||||||
|
* update main class path in build configuration and adjust VCS directory mapping ([7b1f8b1](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7b1f8b117623d327232a1a92a8a44d18582e0189))
|
||||||
|
* update move validation to check for king safety ([#13](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/13)) ([e5e20c5](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/e5e20c566e368b12ca1dc59680c34e9112bf6762))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
|||||||
@@ -138,6 +138,8 @@ tasks.withType(org.gradle.api.tasks.scala.ScalaCompile::class).configureEach {
|
|||||||
exclude("**/resource/GameDtoMapper.scala")
|
exclude("**/resource/GameDtoMapper.scala")
|
||||||
exclude("**/resource/GameResource.scala")
|
exclude("**/resource/GameResource.scala")
|
||||||
exclude("**/redis/GameRedis*.scala")
|
exclude("**/redis/GameRedis*.scala")
|
||||||
|
exclude("**/redis/GameCreationStreamListener.scala")
|
||||||
|
exclude("**/service/GameCreationService.scala")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -2,6 +2,7 @@ package de.nowchess.chess.config
|
|||||||
|
|
||||||
import de.nowchess.api.board.{CastlingRights, Color, File, Piece, PieceType, Rank, Square}
|
import de.nowchess.api.board.{CastlingRights, Color, File, Piece, PieceType, Rank, Square}
|
||||||
import de.nowchess.api.dto.*
|
import de.nowchess.api.dto.*
|
||||||
|
import de.nowchess.api.event.{EventEnvelope, EventType}
|
||||||
import de.nowchess.api.game.{DrawReason, GameContext, GameMode, GameResult}
|
import de.nowchess.api.game.{DrawReason, GameContext, GameMode, GameResult}
|
||||||
import de.nowchess.api.move.{Move, MoveType, PromotionPiece}
|
import de.nowchess.api.move.{Move, MoveType, PromotionPiece}
|
||||||
import de.nowchess.chess.registry.GameCacheDto
|
import de.nowchess.chess.registry.GameCacheDto
|
||||||
@@ -13,6 +14,10 @@ import io.quarkus.runtime.annotations.RegisterForReflection
|
|||||||
classOf[GameCacheDto],
|
classOf[GameCacheDto],
|
||||||
classOf[ClockDto],
|
classOf[ClockDto],
|
||||||
classOf[CreateGameRequestDto],
|
classOf[CreateGameRequestDto],
|
||||||
|
classOf[GameCreationRequestDto],
|
||||||
|
classOf[GameCreationResponseDto],
|
||||||
|
classOf[EventEnvelope],
|
||||||
|
classOf[EventType],
|
||||||
classOf[ErrorEventDto],
|
classOf[ErrorEventDto],
|
||||||
classOf[GameWritebackEventDto],
|
classOf[GameWritebackEventDto],
|
||||||
classOf[GameFullDto],
|
classOf[GameFullDto],
|
||||||
|
|||||||
@@ -418,7 +418,6 @@ class GameEngine(
|
|||||||
val contextBefore = currentContext
|
val contextBefore = currentContext
|
||||||
val nextContext = ruleSet.applyMove(currentContext)(move)
|
val nextContext = ruleSet.applyMove(currentContext)(move)
|
||||||
val captured = computeCaptured(currentContext, move)
|
val captured = computeCaptured(currentContext, move)
|
||||||
val notation = translateMoveToNotation(move, contextBefore.board)
|
|
||||||
currentContext = nextContext
|
currentContext = nextContext
|
||||||
|
|
||||||
advanceClock(contextBefore.turn)
|
advanceClock(contextBefore.turn)
|
||||||
@@ -463,13 +462,18 @@ class GameEngine(
|
|||||||
redoStack = Nil
|
redoStack = Nil
|
||||||
else if status.isCheck then notifyObservers(CheckDetectedEvent(currentContext))
|
else if status.isCheck then notifyObservers(CheckDetectedEvent(currentContext))
|
||||||
|
|
||||||
private def translateMoveToNotation(move: Move, boardBefore: Board): String =
|
private def translateMoveToNotation(move: Move, ctxBefore: GameContext, status: PostMoveStatus): String =
|
||||||
move.moveType match
|
val suffix =
|
||||||
|
if status.isCheckmate then "#"
|
||||||
|
else if status.isCheck then "+"
|
||||||
|
else ""
|
||||||
|
val base = move.moveType match
|
||||||
case MoveType.CastleKingside => "O-O"
|
case MoveType.CastleKingside => "O-O"
|
||||||
case MoveType.CastleQueenside => "O-O-O"
|
case MoveType.CastleQueenside => "O-O-O"
|
||||||
case MoveType.EnPassant => enPassantNotation(move)
|
case MoveType.EnPassant => enPassantNotation(move)
|
||||||
case MoveType.Promotion(pp) => promotionNotation(move, pp)
|
case MoveType.Promotion(pp) => promotionNotation(move, pp)
|
||||||
case MoveType.Normal(isCapture) => normalMoveNotation(move, boardBefore, isCapture)
|
case MoveType.Normal(isCapture) => normalMoveNotation(move, ctxBefore, isCapture)
|
||||||
|
base + suffix
|
||||||
|
|
||||||
private def enPassantNotation(move: Move): String =
|
private def enPassantNotation(move: Move): String =
|
||||||
s"${move.from.file.toString.toLowerCase}x${move.to}"
|
s"${move.from.file.toString.toLowerCase}x${move.to}"
|
||||||
@@ -482,16 +486,31 @@ class GameEngine(
|
|||||||
case PromotionPiece.Knight => "N"
|
case PromotionPiece.Knight => "N"
|
||||||
s"${move.to}=$ppChar"
|
s"${move.to}=$ppChar"
|
||||||
|
|
||||||
private[engine] def normalMoveNotation(move: Move, boardBefore: Board, isCapture: Boolean): String =
|
private[engine] def normalMoveNotation(move: Move, ctxBefore: GameContext, isCapture: Boolean): String =
|
||||||
boardBefore.pieceAt(move.from).map(_.pieceType) match
|
ctxBefore.board.pieceAt(move.from).map(_.pieceType) match
|
||||||
case Some(PieceType.Pawn) =>
|
case Some(PieceType.Pawn) =>
|
||||||
if isCapture then s"${move.from.file.toString.toLowerCase}x${move.to}"
|
if isCapture then s"${move.from.file.toString.toLowerCase}x${move.to}"
|
||||||
else move.to.toString
|
else move.to.toString
|
||||||
case Some(pt) =>
|
case Some(pt) =>
|
||||||
val letter = pieceNotation(pt)
|
val letter = pieceNotation(pt)
|
||||||
if isCapture then s"${letter}x${move.to}" else s"$letter${move.to}"
|
val d = disambiguatePiece(move.from, move.to, pt, ctxBefore)
|
||||||
|
if isCapture then s"$letter${d}x${move.to}" else s"$letter$d${move.to}"
|
||||||
case None => move.to.toString
|
case None => move.to.toString
|
||||||
|
|
||||||
|
private def disambiguatePiece(from: Square, to: Square, pieceType: PieceType, ctx: GameContext): String =
|
||||||
|
if pieceType == PieceType.King then ""
|
||||||
|
else
|
||||||
|
val competitors = ruleSet
|
||||||
|
.allLegalMoves(ctx)
|
||||||
|
.filter(m => m.to == to && m.from != from && ctx.board.pieceAt(m.from).exists(_.pieceType == pieceType))
|
||||||
|
if competitors.isEmpty then ""
|
||||||
|
else
|
||||||
|
val sameFile = competitors.exists(_.from.file == from.file)
|
||||||
|
val sameRank = competitors.exists(_.from.rank == from.rank)
|
||||||
|
if !sameFile then from.file.toString.toLowerCase
|
||||||
|
else if !sameRank then (from.rank.ordinal + 1).toString
|
||||||
|
else from.toString
|
||||||
|
|
||||||
private[engine] def pieceNotation(pieceType: PieceType): String =
|
private[engine] def pieceNotation(pieceType: PieceType): String =
|
||||||
pieceType match
|
pieceType match
|
||||||
case PieceType.Knight => "N"
|
case PieceType.Knight => "N"
|
||||||
@@ -519,9 +538,10 @@ class GameEngine(
|
|||||||
if currentContext.moves.isEmpty then
|
if currentContext.moves.isEmpty then
|
||||||
notifyObservers(InvalidMoveEvent(currentContext, InvalidMoveReason.NothingToUndo))
|
notifyObservers(InvalidMoveEvent(currentContext, InvalidMoveReason.NothingToUndo))
|
||||||
else
|
else
|
||||||
val lastMove = currentContext.moves.last
|
val lastMove = currentContext.moves.last
|
||||||
val prevCtx = replayContextFromMoves(currentContext.moves.dropRight(1))
|
val prevCtx = replayContextFromMoves(currentContext.moves.dropRight(1))
|
||||||
val notation = translateMoveToNotation(lastMove, prevCtx.board)
|
val postStatus = ruleSet.postMoveStatus(currentContext)
|
||||||
|
val notation = translateMoveToNotation(lastMove, prevCtx, postStatus)
|
||||||
redoStack = lastMove :: redoStack
|
redoStack = lastMove :: redoStack
|
||||||
currentContext = prevCtx
|
currentContext = prevCtx
|
||||||
notifyObservers(MoveUndoneEvent(currentContext, notation))
|
notifyObservers(MoveUndoneEvent(currentContext, notation))
|
||||||
|
|||||||
@@ -0,0 +1,146 @@
|
|||||||
|
package de.nowchess.chess.redis
|
||||||
|
|
||||||
|
import com.fasterxml.jackson.databind.ObjectMapper
|
||||||
|
import de.nowchess.api.dto.{GameCreationRequestDto, GameCreationResponseDto}
|
||||||
|
import de.nowchess.api.event.{EventEnvelope, EventType}
|
||||||
|
import de.nowchess.chess.config.RedisConfig
|
||||||
|
import de.nowchess.chess.service.GameCreationService
|
||||||
|
import io.quarkus.redis.datasource.RedisDataSource
|
||||||
|
import io.quarkus.redis.datasource.stream.{StreamMessage, XAddArgs, XGroupCreateArgs, XReadGroupArgs}
|
||||||
|
import io.quarkus.runtime.StartupEvent
|
||||||
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
|
import jakarta.enterprise.event.Observes
|
||||||
|
import jakarta.inject.Inject
|
||||||
|
import org.eclipse.microprofile.config.inject.ConfigProperty
|
||||||
|
import org.eclipse.microprofile.context.ManagedExecutor
|
||||||
|
import org.jboss.logging.Logger
|
||||||
|
import scala.compiletime.uninitialized
|
||||||
|
import scala.jdk.CollectionConverters.*
|
||||||
|
import scala.util.{Failure, Success, Try}
|
||||||
|
import java.time.Duration
|
||||||
|
import java.util.UUID
|
||||||
|
|
||||||
|
@ApplicationScoped
|
||||||
|
class GameCreationStreamListener:
|
||||||
|
|
||||||
|
// scalafix:off DisableSyntax.var
|
||||||
|
@Inject var redis: RedisDataSource = uninitialized
|
||||||
|
@Inject var objectMapper: ObjectMapper = uninitialized
|
||||||
|
@Inject var creationService: GameCreationService = uninitialized
|
||||||
|
@Inject var executor: ManagedExecutor = uninitialized
|
||||||
|
@Inject var redisConfig: RedisConfig = uninitialized
|
||||||
|
@ConfigProperty(name = "nowchess.game-creation-stream.enabled", defaultValue = "true")
|
||||||
|
private var streamEnabled: Boolean = true
|
||||||
|
// scalafix:on DisableSyntax.var
|
||||||
|
|
||||||
|
private val log = Logger.getLogger(classOf[GameCreationStreamListener])
|
||||||
|
private val groupName = "core-game-creation"
|
||||||
|
private val consumerId = UUID.randomUUID().toString
|
||||||
|
private val maxRetries = 3
|
||||||
|
private val maxStreamLen = 1000L
|
||||||
|
|
||||||
|
private def requestStream: String = s"${redisConfig.prefix}:game-creation"
|
||||||
|
private def responseStream: String = s"${redisConfig.prefix}:game-creation-response"
|
||||||
|
private def dlqStream: String = s"${redisConfig.prefix}:dlq"
|
||||||
|
|
||||||
|
def start(@Observes _ev: StartupEvent): Unit =
|
||||||
|
if streamEnabled then
|
||||||
|
createGroupIfAbsent()
|
||||||
|
executor.submit(
|
||||||
|
new Runnable:
|
||||||
|
def run(): Unit = pollLoop(),
|
||||||
|
)
|
||||||
|
log.infof("Game-creation request listener started (consumer=%s)", consumerId)
|
||||||
|
|
||||||
|
private def createGroupIfAbsent(): Unit =
|
||||||
|
Try(
|
||||||
|
redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xgroupCreate(requestStream, groupName, "0", new XGroupCreateArgs().mkstream()),
|
||||||
|
) match
|
||||||
|
case Failure(ex) if Option(ex.getMessage).exists(_.contains("BUSYGROUP")) => ()
|
||||||
|
case Failure(ex) => log.warnf(ex, "Failed to create game-creation consumer group")
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def pollLoop(): Unit =
|
||||||
|
while true do
|
||||||
|
Try {
|
||||||
|
val messages = redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xreadgroup(
|
||||||
|
groupName,
|
||||||
|
consumerId,
|
||||||
|
requestStream,
|
||||||
|
">",
|
||||||
|
new XReadGroupArgs().count(10).block(Duration.ofSeconds(2)),
|
||||||
|
)
|
||||||
|
Option(messages).foreach(_.forEach(handleMessage))
|
||||||
|
} match
|
||||||
|
case Failure(ex) => log.warnf(ex, "Error in game-creation poll loop")
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def handleMessage(msg: StreamMessage[String, String, String]): Unit =
|
||||||
|
val json = msg.payload().get("data")
|
||||||
|
val attempt = Option(msg.payload().get("attempt")).flatMap(_.toIntOption).getOrElse(0)
|
||||||
|
Try(objectMapper.readValue(json, classOf[EventEnvelope])) match
|
||||||
|
case Failure(ex) =>
|
||||||
|
log.errorf(ex, "Unparseable game-creation event, sending to DLQ: %s", json)
|
||||||
|
toDlq(EventType.GameCreationRequest.toString, json, ex, attempt)
|
||||||
|
ack(msg.id())
|
||||||
|
case Success(envelope) =>
|
||||||
|
processEnvelope(msg, envelope, json, attempt)
|
||||||
|
|
||||||
|
private def processEnvelope(
|
||||||
|
msg: StreamMessage[String, String, String],
|
||||||
|
envelope: EventEnvelope,
|
||||||
|
json: String,
|
||||||
|
attempt: Int,
|
||||||
|
): Unit =
|
||||||
|
Try {
|
||||||
|
val req = objectMapper.treeToValue(envelope.payload, classOf[GameCreationRequestDto])
|
||||||
|
val entry = creationService.createGame(req)
|
||||||
|
publishResponse(envelope.correlationId, GameCreationResponseDto(Some(entry.gameId)))
|
||||||
|
} match
|
||||||
|
case Success(_) => ack(msg.id())
|
||||||
|
case Failure(ex) if attempt + 1 < maxRetries =>
|
||||||
|
log.warnf(ex, "Game creation failed (attempt %d), retrying", attempt)
|
||||||
|
retry(json, attempt + 1)
|
||||||
|
ack(msg.id())
|
||||||
|
case Failure(ex) =>
|
||||||
|
log.errorf(ex, "Game creation failed after %d attempts, sending to DLQ", maxRetries)
|
||||||
|
publishResponse(envelope.correlationId, GameCreationResponseDto(None, Some("Game creation failed")))
|
||||||
|
toDlq(envelope.`type`.toString, json, ex, attempt)
|
||||||
|
ack(msg.id())
|
||||||
|
|
||||||
|
private def publishResponse(correlationId: Option[String], resp: GameCreationResponseDto): Unit =
|
||||||
|
val payload = objectMapper.valueToTree[com.fasterxml.jackson.databind.JsonNode](resp)
|
||||||
|
val envelope = EventEnvelope.of(EventType.GameCreationResponse, payload, correlationId)
|
||||||
|
xadd(responseStream, Map("data" -> objectMapper.writeValueAsString(envelope)))
|
||||||
|
|
||||||
|
private def retry(json: String, attempt: Int): Unit =
|
||||||
|
xadd(requestStream, Map("data" -> json, "attempt" -> attempt.toString))
|
||||||
|
|
||||||
|
private def toDlq(eventType: String, json: String, error: Throwable, attempt: Int): Unit =
|
||||||
|
xadd(
|
||||||
|
dlqStream,
|
||||||
|
Map(
|
||||||
|
"data" -> json,
|
||||||
|
"eventType" -> eventType,
|
||||||
|
"error" -> Option(error.getMessage).getOrElse(error.getClass.getName),
|
||||||
|
"attempt" -> (attempt + 1).toString,
|
||||||
|
),
|
||||||
|
)
|
||||||
|
|
||||||
|
private def ack(id: String): Unit =
|
||||||
|
Try(redis.stream(classOf[String]).xack(requestStream, groupName, id)) match
|
||||||
|
case Failure(ex) => log.warnf(ex, "Failed to ack message %s", id)
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def xadd(key: String, fields: Map[String, String]): Unit =
|
||||||
|
Try(
|
||||||
|
redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xadd(key, new XAddArgs().maxlen(maxStreamLen).nearlyExactTrimming(), fields.asJava),
|
||||||
|
) match
|
||||||
|
case Failure(ex) => log.errorf(ex, "Failed to publish to stream %s", key)
|
||||||
|
case Success(_) => ()
|
||||||
@@ -0,0 +1,71 @@
|
|||||||
|
package de.nowchess.chess.service
|
||||||
|
|
||||||
|
import de.nowchess.api.dto.{GameCreationRequestDto, PlayerInfoDto, TimeControlDto}
|
||||||
|
import de.nowchess.api.game.{GameContext, GameMode, TimeControl}
|
||||||
|
import de.nowchess.api.player.{PlayerId, PlayerInfo}
|
||||||
|
import de.nowchess.chess.engine.GameEngine
|
||||||
|
import de.nowchess.chess.grpc.RuleSetGrpcAdapter
|
||||||
|
import de.nowchess.chess.redis.GameRedisSubscriberManager
|
||||||
|
import de.nowchess.chess.registry.{GameEntry, GameRegistry}
|
||||||
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
|
import jakarta.inject.Inject
|
||||||
|
import org.jboss.logging.Logger
|
||||||
|
import scala.compiletime.uninitialized
|
||||||
|
|
||||||
|
@ApplicationScoped
|
||||||
|
class GameCreationService:
|
||||||
|
|
||||||
|
private val log = Logger.getLogger(classOf[GameCreationService])
|
||||||
|
|
||||||
|
// scalafix:off DisableSyntax.var
|
||||||
|
@Inject var registry: GameRegistry = uninitialized
|
||||||
|
@Inject var ruleSetAdapter: RuleSetGrpcAdapter = uninitialized
|
||||||
|
@Inject var subscriberManager: GameRedisSubscriberManager = uninitialized
|
||||||
|
// scalafix:on DisableSyntax.var
|
||||||
|
|
||||||
|
private val DefaultWhite = PlayerInfo(PlayerId("p1"), "Player 1")
|
||||||
|
private val DefaultBlack = PlayerInfo(PlayerId("p2"), "Player 2")
|
||||||
|
|
||||||
|
def createGame(req: GameCreationRequestDto): GameEntry =
|
||||||
|
val white = playerInfoFrom(req.white, DefaultWhite)
|
||||||
|
val black = playerInfoFrom(req.black, DefaultBlack)
|
||||||
|
val tc = toTimeControl(req.timeControl)
|
||||||
|
val mode = req.mode.getOrElse(GameMode.Open)
|
||||||
|
val entry = newEntry(GameContext.initial, white, black, tc, mode)
|
||||||
|
registry.store(entry)
|
||||||
|
subscriberManager.subscribeGame(entry.gameId)
|
||||||
|
log.infof(
|
||||||
|
"Game %s created — white=%s black=%s mode=%s",
|
||||||
|
entry.gameId,
|
||||||
|
white.displayName,
|
||||||
|
black.displayName,
|
||||||
|
mode.toString,
|
||||||
|
)
|
||||||
|
entry
|
||||||
|
|
||||||
|
private def playerInfoFrom(dto: Option[PlayerInfoDto], default: PlayerInfo): PlayerInfo =
|
||||||
|
dto.fold(default)(d => PlayerInfo(PlayerId(d.id), d.displayName))
|
||||||
|
|
||||||
|
private def toTimeControl(dto: Option[TimeControlDto]): TimeControl =
|
||||||
|
dto match
|
||||||
|
case None => TimeControl.Unlimited
|
||||||
|
case Some(tc) =>
|
||||||
|
tc.daysPerMove match
|
||||||
|
case Some(d) => TimeControl.Correspondence(d)
|
||||||
|
case None =>
|
||||||
|
tc.limitSeconds.fold(TimeControl.Unlimited)(l => TimeControl.Clock(l, tc.incrementSeconds.getOrElse(0)))
|
||||||
|
|
||||||
|
private def newEntry(
|
||||||
|
ctx: GameContext,
|
||||||
|
white: PlayerInfo,
|
||||||
|
black: PlayerInfo,
|
||||||
|
tc: TimeControl,
|
||||||
|
mode: GameMode,
|
||||||
|
): GameEntry =
|
||||||
|
GameEntry(
|
||||||
|
registry.generateId(),
|
||||||
|
GameEngine(initialContext = ctx, ruleSet = ruleSetAdapter, timeControl = tc),
|
||||||
|
white,
|
||||||
|
black,
|
||||||
|
mode = mode,
|
||||||
|
)
|
||||||
@@ -18,6 +18,8 @@ nowchess:
|
|||||||
enabled: false
|
enabled: false
|
||||||
coordinator:
|
coordinator:
|
||||||
enabled: false
|
enabled: false
|
||||||
|
game-creation-stream:
|
||||||
|
enabled: false
|
||||||
redis:
|
redis:
|
||||||
host: localhost
|
host: localhost
|
||||||
port: 6379
|
port: 6379
|
||||||
|
|||||||
+2
-2
@@ -1,6 +1,6 @@
|
|||||||
package de.nowchess.chess.engine
|
package de.nowchess.chess.engine
|
||||||
|
|
||||||
import de.nowchess.api.board.{Board, Color, File, PieceType, Rank, Square}
|
import de.nowchess.api.board.{Color, File, PieceType, Rank, Square}
|
||||||
import de.nowchess.api.game.GameContext
|
import de.nowchess.api.game.GameContext
|
||||||
import de.nowchess.api.move.{Move, MoveType, PromotionPiece}
|
import de.nowchess.api.move.{Move, MoveType, PromotionPiece}
|
||||||
import de.nowchess.chess.observer.{GameEvent, InvalidMoveEvent, InvalidMoveReason, Observer}
|
import de.nowchess.chess.observer.{GameEvent, InvalidMoveEvent, InvalidMoveReason, Observer}
|
||||||
@@ -137,7 +137,7 @@ class GameEngineIntegrationTest extends AnyFunSuite with Matchers:
|
|||||||
|
|
||||||
test("normalMoveNotation handles missing source piece"):
|
test("normalMoveNotation handles missing source piece"):
|
||||||
val engine = new GameEngine(ruleSet = DefaultRules)
|
val engine = new GameEngine(ruleSet = DefaultRules)
|
||||||
val result = engine.normalMoveNotation(Move(sq("e3"), sq("e4")), Board.initial, isCapture = false)
|
val result = engine.normalMoveNotation(Move(sq("e3"), sq("e4")), GameContext.initial, isCapture = false)
|
||||||
|
|
||||||
result shouldBe "e4"
|
result shouldBe "e4"
|
||||||
|
|
||||||
|
|||||||
@@ -106,7 +106,7 @@ class GameEngineNotationTest extends AnyFunSuite with Matchers:
|
|||||||
engine.undo()
|
engine.undo()
|
||||||
|
|
||||||
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
||||||
evt.pgnNotation shouldBe "e8=B"
|
evt.pgnNotation shouldBe "e8=B+"
|
||||||
|
|
||||||
// ── King normal move notation (line 246) ───────────────────────────
|
// ── King normal move notation (line 246) ───────────────────────────
|
||||||
|
|
||||||
@@ -134,3 +134,87 @@ class GameEngineNotationTest extends AnyFunSuite with Matchers:
|
|||||||
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
||||||
evt.pgnNotation should startWith("K")
|
evt.pgnNotation should startWith("K")
|
||||||
evt.pgnNotation should include("f1")
|
evt.pgnNotation should include("f1")
|
||||||
|
|
||||||
|
// ── Disambiguation: two knights same rank (file suffix) ────────────
|
||||||
|
|
||||||
|
test("undo with two knights on same rank disambiguates by file"):
|
||||||
|
// White knights on b1 and f3, black pawn on h7 prevents draw, both knights can reach d2
|
||||||
|
val board = FenParser.parseBoard("k7/7p/8/8/8/5N2/8/1N5K").get
|
||||||
|
val ctx = GameContext.initial
|
||||||
|
.withBoard(board)
|
||||||
|
.withTurn(Color.White)
|
||||||
|
.withCastlingRights(de.nowchess.api.board.CastlingRights(false, false, false, false))
|
||||||
|
|
||||||
|
val engine = new GameEngine(ctx, DefaultRules)
|
||||||
|
val events = captureEvents(engine)
|
||||||
|
|
||||||
|
// Knight on b1 moves to d2; notation must be "Nbd2" to disambiguate from Nf3
|
||||||
|
engine.processUserInput("b1d2")
|
||||||
|
events.clear()
|
||||||
|
engine.undo()
|
||||||
|
|
||||||
|
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
||||||
|
evt.pgnNotation should startWith("Nb")
|
||||||
|
evt.pgnNotation should include("d2")
|
||||||
|
|
||||||
|
// ── Disambiguation: two knights same file (rank suffix) ────────────
|
||||||
|
|
||||||
|
test("undo with two knights on same file disambiguates by rank"):
|
||||||
|
// White knights on b1 and b3, both can reach d2 or c5; use b1->d2
|
||||||
|
val board = FenParser.parseBoard("k7/7p/8/8/8/1N6/8/1N5K").get
|
||||||
|
val ctx = GameContext.initial
|
||||||
|
.withBoard(board)
|
||||||
|
.withTurn(Color.White)
|
||||||
|
.withCastlingRights(de.nowchess.api.board.CastlingRights(false, false, false, false))
|
||||||
|
|
||||||
|
val engine = new GameEngine(ctx, DefaultRules)
|
||||||
|
val events = captureEvents(engine)
|
||||||
|
|
||||||
|
// Knight on b1 moves to d2; notation must be "N1d2" to disambiguate from b3
|
||||||
|
engine.processUserInput("b1d2")
|
||||||
|
events.clear()
|
||||||
|
engine.undo()
|
||||||
|
|
||||||
|
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
||||||
|
evt.pgnNotation should include("1")
|
||||||
|
evt.pgnNotation should include("d2")
|
||||||
|
|
||||||
|
// ── Check suffix (+) ───────────────────────────────────────────────
|
||||||
|
|
||||||
|
test("undo after move that gives check emits notation with + suffix"):
|
||||||
|
// White rook a1, white king h1, black king e8; Ra1-e1 gives check on e-file
|
||||||
|
val board = FenParser.parseBoard("4k3/8/8/8/8/8/8/R6K").get
|
||||||
|
val ctx = GameContext.initial
|
||||||
|
.withBoard(board)
|
||||||
|
.withTurn(Color.White)
|
||||||
|
.withCastlingRights(de.nowchess.api.board.CastlingRights(false, false, false, false))
|
||||||
|
|
||||||
|
val engine = new GameEngine(ctx, DefaultRules)
|
||||||
|
val events = captureEvents(engine)
|
||||||
|
|
||||||
|
engine.processUserInput("a1e1")
|
||||||
|
events.clear()
|
||||||
|
engine.undo()
|
||||||
|
|
||||||
|
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
||||||
|
evt.pgnNotation should endWith("+")
|
||||||
|
|
||||||
|
// ── Checkmate suffix (#) ──────────────────────────────────────────
|
||||||
|
|
||||||
|
test("undo after checkmate move emits notation with # suffix"):
|
||||||
|
// Fool's mate setup (before final move): 1.f3 e5 2.g4 -- black plays Qd8-h4#
|
||||||
|
val board = FenParser.parseBoard("rnbqkbnr/pppp1ppp/8/4p3/6P1/5P2/PPPPP2P/RNBQKBNR").get
|
||||||
|
val ctx = GameContext.initial
|
||||||
|
.withBoard(board)
|
||||||
|
.withTurn(Color.Black)
|
||||||
|
.withCastlingRights(de.nowchess.api.board.CastlingRights(true, true, true, true))
|
||||||
|
|
||||||
|
val engine = new GameEngine(ctx, DefaultRules)
|
||||||
|
val events = captureEvents(engine)
|
||||||
|
|
||||||
|
engine.processUserInput("d8h4")
|
||||||
|
events.clear()
|
||||||
|
engine.undo()
|
||||||
|
|
||||||
|
val evt = events.collect { case e: MoveUndoneEvent => e }.head
|
||||||
|
evt.pgnNotation should endWith("#")
|
||||||
|
|||||||
@@ -0,0 +1,68 @@
|
|||||||
|
package de.nowchess.chess.service
|
||||||
|
|
||||||
|
import de.nowchess.api.dto.{GameCreationRequestDto, PlayerInfoDto, TimeControlDto}
|
||||||
|
import de.nowchess.api.game.{GameMode, TimeControl}
|
||||||
|
import de.nowchess.api.player.PlayerType
|
||||||
|
import de.nowchess.chess.client.CombinedExportResponse
|
||||||
|
import de.nowchess.chess.grpc.IoGrpcClientWrapper
|
||||||
|
import de.nowchess.chess.redis.GameRedisSubscriberManager
|
||||||
|
import io.quarkus.test.InjectMock
|
||||||
|
import io.quarkus.test.junit.QuarkusTest
|
||||||
|
import jakarta.inject.Inject
|
||||||
|
import org.junit.jupiter.api.Assertions.*
|
||||||
|
import org.junit.jupiter.api.{BeforeEach, DisplayName, Test}
|
||||||
|
import org.mockito.ArgumentMatchers.any
|
||||||
|
import org.mockito.Mockito.{verify, when}
|
||||||
|
import scala.compiletime.uninitialized
|
||||||
|
|
||||||
|
// scalafix:off
|
||||||
|
@QuarkusTest
|
||||||
|
@DisplayName("GameCreationService")
|
||||||
|
class GameCreationServiceTest:
|
||||||
|
|
||||||
|
@Inject
|
||||||
|
var service: GameCreationService = uninitialized
|
||||||
|
|
||||||
|
@InjectMock
|
||||||
|
var subscriberManager: GameRedisSubscriberManager = uninitialized
|
||||||
|
|
||||||
|
@InjectMock
|
||||||
|
var ioWrapper: IoGrpcClientWrapper = uninitialized
|
||||||
|
|
||||||
|
@BeforeEach
|
||||||
|
def setup(): Unit =
|
||||||
|
when(ioWrapper.exportCombined(any()))
|
||||||
|
.thenReturn(CombinedExportResponse("rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1", ""))
|
||||||
|
|
||||||
|
private def player(id: String, name: String): PlayerInfoDto =
|
||||||
|
PlayerInfoDto(id, name, PlayerType.Human)
|
||||||
|
|
||||||
|
@Test
|
||||||
|
def createsGameAndSubscribes(): Unit =
|
||||||
|
val req =
|
||||||
|
GameCreationRequestDto(Some(player("w", "White")), Some(player("b", "Black")), None, Some(GameMode.Authenticated))
|
||||||
|
val entry = service.createGame(req)
|
||||||
|
assertNotNull(entry.gameId)
|
||||||
|
assertEquals("White", entry.white.displayName)
|
||||||
|
assertEquals("Black", entry.black.displayName)
|
||||||
|
assertEquals(GameMode.Authenticated, entry.mode)
|
||||||
|
verify(subscriberManager).subscribeGame(entry.gameId)
|
||||||
|
|
||||||
|
@Test
|
||||||
|
def defaultsToOpenModeAndDefaultPlayers(): Unit =
|
||||||
|
val entry = service.createGame(GameCreationRequestDto(None, None, None, None))
|
||||||
|
assertEquals(GameMode.Open, entry.mode)
|
||||||
|
assertEquals("Player 1", entry.white.displayName)
|
||||||
|
assertEquals("Player 2", entry.black.displayName)
|
||||||
|
|
||||||
|
@Test
|
||||||
|
def mapsClockTimeControl(): Unit =
|
||||||
|
val tc = TimeControlDto(Some(300), Some(5), None)
|
||||||
|
val entry = service.createGame(GameCreationRequestDto(None, None, Some(tc), None))
|
||||||
|
assertEquals(TimeControl.Clock(300, 5), entry.engine.timeControl)
|
||||||
|
|
||||||
|
@Test
|
||||||
|
def mapsCorrespondenceTimeControl(): Unit =
|
||||||
|
val tc = TimeControlDto(None, None, Some(3))
|
||||||
|
val entry = service.createGame(GameCreationRequestDto(None, None, Some(tc), None))
|
||||||
|
assertEquals(TimeControl.Correspondence(3), entry.engine.timeControl)
|
||||||
@@ -1,3 +1,3 @@
|
|||||||
MAJOR=0
|
MAJOR=0
|
||||||
MINOR=46
|
MINOR=48
|
||||||
PATCH=0
|
PATCH=0
|
||||||
|
|||||||
@@ -296,3 +296,31 @@
|
|||||||
|
|
||||||
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-02)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* add authentication permissions for metrics endpoints in application.yml ([04edd4d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/04edd4d6fd8a63196c36f6d67992832febc9bebb))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* NCS-14 implemented insufficient moves rule ([#30](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/30)) ([b0399a4](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b0399a4e489950083066c9538df9a84dcc7a4613))
|
||||||
|
* NCS-25 Add linters to keep quality up ([#27](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/27)) ([fd4e67d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/fd4e67d4f782a7e955822d90cb909d0a81676fb2))
|
||||||
|
* NCS-29 JSON - Cherry Picked ([#28](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/28)) ([dbcafd2](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/dbcafd286993e0604a6fa286c5543581a149439e))
|
||||||
|
* NCS-30 FEN Parser using ParserCombinators ([#21](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/21)) ([b4bc72f](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b4bc72f7e49f94d6e1bc805c68680e5fe8ef8e36))
|
||||||
|
* NCS-31 FastParse FEN ([#22](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/22)) ([7a045d3](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7a045d31d757bbc5aa6f4bad2664ebe8b8519cac))
|
||||||
|
* NCS-37 Quarkus integration ([#35](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/35)) ([f088c4e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/f088c4e9ffcc498d3d1b6f01e8f50042d5830d55))
|
||||||
|
* NCS-41 Bot Platform ([#33](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/33)) ([8744bee](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8744bee2dd20966dae90a09c21a43d5b06f59e00))
|
||||||
|
* NCS-53 changed IO to MicroService for easier scaling ([#37](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/37)) ([b5a2966](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b5a2966adafa9650f0f7d601bdeb8fdd13710327))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
* update application.yml with new API root paths and add Micrometer and OpenTelemetry dependencies ([72ce262](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/72ce262bc491f94297700e6002fb5d0812e2cc2a))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* IO microservice ([#38](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/38)) ([a386f57](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a386f57c21d34ead6cc6f92836c52b714597e289))
|
||||||
|
* **pgn:** add SAN disambiguation and check/checkmate suffixes [NCS-42] ([#56](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/56)) ([2579539](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2579539084152178f4482ddb7b84b7f1162f10da))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "feat: add authentication permissions for metrics endpoints in application.yml" ([a298417](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a298417b9e4d68dc73bbf40be63d9484536e9f83))
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
|||||||
@@ -31,7 +31,9 @@ object PgnExporter extends GameContextExport:
|
|||||||
if moves.isEmpty then ""
|
if moves.isEmpty then ""
|
||||||
else
|
else
|
||||||
val contexts = moves.scanLeft(GameContext.initial)((ctx, move) => DefaultRules.applyMove(ctx)(move))
|
val contexts = moves.scanLeft(GameContext.initial)((ctx, move) => DefaultRules.applyMove(ctx)(move))
|
||||||
val sanMoves = moves.zip(contexts).map { case (move, ctx) => moveToAlgebraic(move, ctx.board) }
|
val sanMoves = moves.zip(contexts).zip(contexts.tail).map { case ((move, ctxBefore), ctxAfter) =>
|
||||||
|
moveToAlgebraic(move, ctxBefore, ctxAfter)
|
||||||
|
}
|
||||||
|
|
||||||
val groupedMoves = sanMoves.zipWithIndex.groupBy(_._2 / 2)
|
val groupedMoves = sanMoves.zipWithIndex.groupBy(_._2 / 2)
|
||||||
val moveLines = for (moveNumber, movePairs) <- groupedMoves.toList.sortBy(_._1) yield
|
val moveLines = for (moveNumber, movePairs) <- groupedMoves.toList.sortBy(_._1) yield
|
||||||
@@ -48,9 +50,24 @@ object PgnExporter extends GameContextExport:
|
|||||||
else if moveText.isEmpty then headerLines
|
else if moveText.isEmpty then headerLines
|
||||||
else s"$headerLines\n\n$moveText"
|
else s"$headerLines\n\n$moveText"
|
||||||
|
|
||||||
/** Convert a Move to Standard Algebraic Notation using the board state before the move. */
|
private def disambiguate(from: Square, to: Square, pieceType: PieceType, ctx: GameContext): String =
|
||||||
private def moveToAlgebraic(move: Move, boardBefore: Board): String =
|
val competitors = DefaultRules
|
||||||
move.moveType match
|
.allLegalMoves(ctx)
|
||||||
|
.filter(m => m.to == to && m.from != from && ctx.board.pieceAt(m.from).exists(_.pieceType == pieceType))
|
||||||
|
if competitors.isEmpty then ""
|
||||||
|
else
|
||||||
|
val sameFile = competitors.exists(_.from.file == from.file)
|
||||||
|
val sameRank = competitors.exists(_.from.rank == from.rank)
|
||||||
|
if !sameFile then from.file.toString.toLowerCase
|
||||||
|
else if !sameRank then (from.rank.ordinal + 1).toString
|
||||||
|
else from.toString
|
||||||
|
|
||||||
|
private def moveToAlgebraic(move: Move, ctxBefore: GameContext, ctxAfter: GameContext): String =
|
||||||
|
val suffix =
|
||||||
|
if DefaultRules.isCheckmate(ctxAfter) then "#"
|
||||||
|
else if DefaultRules.isCheck(ctxAfter) then "+"
|
||||||
|
else ""
|
||||||
|
val base = move.moveType match
|
||||||
case MoveType.CastleKingside => "O-O"
|
case MoveType.CastleKingside => "O-O"
|
||||||
case MoveType.CastleQueenside => "O-O-O"
|
case MoveType.CastleQueenside => "O-O-O"
|
||||||
case MoveType.EnPassant => s"${move.from.file.toString.toLowerCase}x${move.to}"
|
case MoveType.EnPassant => s"${move.from.file.toString.toLowerCase}x${move.to}"
|
||||||
@@ -60,18 +77,19 @@ object PgnExporter extends GameContextExport:
|
|||||||
case PromotionPiece.Rook => "=R"
|
case PromotionPiece.Rook => "=R"
|
||||||
case PromotionPiece.Bishop => "=B"
|
case PromotionPiece.Bishop => "=B"
|
||||||
case PromotionPiece.Knight => "=N"
|
case PromotionPiece.Knight => "=N"
|
||||||
val isCapture = boardBefore.pieceAt(move.to).isDefined
|
val isCapture = ctxBefore.board.pieceAt(move.to).isDefined
|
||||||
if isCapture then s"${move.from.file.toString.toLowerCase}x${move.to}$promSuffix"
|
if isCapture then s"${move.from.file.toString.toLowerCase}x${move.to}$promSuffix"
|
||||||
else s"${move.to}$promSuffix"
|
else s"${move.to}$promSuffix"
|
||||||
case MoveType.Normal(isCapture) =>
|
case MoveType.Normal(isCapture) =>
|
||||||
val dest = move.to.toString
|
val dest = move.to.toString
|
||||||
val capStr = if isCapture then "x" else ""
|
val capStr = if isCapture then "x" else ""
|
||||||
boardBefore.pieceAt(move.from).map(_.pieceType).getOrElse(PieceType.Pawn) match
|
ctxBefore.board.pieceAt(move.from).map(_.pieceType).getOrElse(PieceType.Pawn) match
|
||||||
case PieceType.Pawn =>
|
case PieceType.Pawn =>
|
||||||
if isCapture then s"${move.from.file.toString.toLowerCase}x$dest"
|
if isCapture then s"${move.from.file.toString.toLowerCase}x$dest"
|
||||||
else dest
|
else dest
|
||||||
case PieceType.Knight => s"N$capStr$dest"
|
case PieceType.Knight => s"N${disambiguate(move.from, move.to, PieceType.Knight, ctxBefore)}$capStr$dest"
|
||||||
case PieceType.Bishop => s"B$capStr$dest"
|
case PieceType.Bishop => s"B${disambiguate(move.from, move.to, PieceType.Bishop, ctxBefore)}$capStr$dest"
|
||||||
case PieceType.Rook => s"R$capStr$dest"
|
case PieceType.Rook => s"R${disambiguate(move.from, move.to, PieceType.Rook, ctxBefore)}$capStr$dest"
|
||||||
case PieceType.Queen => s"Q$capStr$dest"
|
case PieceType.Queen => s"Q${disambiguate(move.from, move.to, PieceType.Queen, ctxBefore)}$capStr$dest"
|
||||||
case PieceType.King => s"K$capStr$dest"
|
case PieceType.King => s"K$capStr$dest"
|
||||||
|
base + suffix
|
||||||
|
|||||||
@@ -112,3 +112,38 @@ class PgnExporterTest extends AnyFunSuite with Matchers:
|
|||||||
pgn should include("exf8=Q")
|
pgn should include("exf8=Q")
|
||||||
pawnCapturePgn should include("exd3")
|
pawnCapturePgn should include("exd3")
|
||||||
quietPromotionPgn should include("e8=Q")
|
quietPromotionPgn should include("e8=Q")
|
||||||
|
|
||||||
|
test("exportGame disambiguates when two knights can reach same square"):
|
||||||
|
// 1.Nf3 a6 2.d3 a5 3.Nfd2: d3 clears d2 so both Nb1 and Nf3 can reach d2; must emit "Nfd2"
|
||||||
|
val moves = List(
|
||||||
|
Move(sq("g1"), sq("f3")),
|
||||||
|
Move(sq("a7"), sq("a6")),
|
||||||
|
Move(sq("d2"), sq("d3")),
|
||||||
|
Move(sq("a6"), sq("a5")),
|
||||||
|
Move(sq("f3"), sq("d2")),
|
||||||
|
)
|
||||||
|
val pgn = PgnExporter.exportGame(Map.empty, moves)
|
||||||
|
pgn should include("Nfd2")
|
||||||
|
|
||||||
|
test("exportGame appends + after move that gives check"):
|
||||||
|
// 1.e4 e5 2.Qh5 Nc6 3.Qxf7+ — queen captures f7, gives check to black king on e8
|
||||||
|
val moves = List(
|
||||||
|
Move(sq("e2"), sq("e4")),
|
||||||
|
Move(sq("e7"), sq("e5")),
|
||||||
|
Move(sq("d1"), sq("h5")),
|
||||||
|
Move(sq("b8"), sq("c6")),
|
||||||
|
Move(sq("h5"), sq("f7"), MoveType.Normal(isCapture = true)),
|
||||||
|
)
|
||||||
|
val pgn = PgnExporter.exportGame(Map.empty, moves)
|
||||||
|
pgn should include("Qxf7+")
|
||||||
|
|
||||||
|
test("exportGame appends # after checkmate move"):
|
||||||
|
// Fool's mate: 1.f3 e5 2.g4 Qh4#
|
||||||
|
val moves = List(
|
||||||
|
Move(sq("f2"), sq("f3")),
|
||||||
|
Move(sq("e7"), sq("e5")),
|
||||||
|
Move(sq("g2"), sq("g4")),
|
||||||
|
Move(sq("d8"), sq("h4")),
|
||||||
|
)
|
||||||
|
val pgn = PgnExporter.exportGame(Map("Result" -> "*"), moves)
|
||||||
|
pgn should include("Qh4#")
|
||||||
|
|||||||
@@ -1,3 +1,3 @@
|
|||||||
MAJOR=0
|
MAJOR=0
|
||||||
MINOR=22
|
MINOR=23
|
||||||
PATCH=0
|
PATCH=0
|
||||||
|
|||||||
@@ -156,3 +156,50 @@
|
|||||||
### Reverts
|
### Reverts
|
||||||
|
|
||||||
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-03)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* **docker:** add .dockerignore and .gitignore files for build exclusions ([c987d8e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c987d8e258c0e6c4cfbdaa8381c64c410d7a2b83))
|
||||||
|
* **docker:** add Dockerfiles for building Quarkus application in native and JVM modes ([3f2d2bb](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f2d2bb4c97fa8cddba66e1da4427c54236dfeed))
|
||||||
|
* **docker:** add Dockerfiles for Quarkus application in JVM and native modes ([34b9933](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/34b993304670cf2aa62cd2f6460cee7b9864b08e))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#46](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/46)) ([649566e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/649566eb3fcf38f91c8896a739f74ea318af312d))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#47](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/47)) ([87dfc6c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87dfc6c2bcce7f7d58fc641bd8d468a2e584c108))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* **official-bots:** NCS-70-auto-register official bots with account service ([#59](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/59)) ([7117a93](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7117a93376272094d0b1a6abf2121254ce396684))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
## (2026-06-09)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* **docker:** add .dockerignore and .gitignore files for build exclusions ([c987d8e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c987d8e258c0e6c4cfbdaa8381c64c410d7a2b83))
|
||||||
|
* **docker:** add Dockerfiles for building Quarkus application in native and JVM modes ([3f2d2bb](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f2d2bb4c97fa8cddba66e1da4427c54236dfeed))
|
||||||
|
* **docker:** add Dockerfiles for Quarkus application in JVM and native modes ([34b9933](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/34b993304670cf2aa62cd2f6460cee7b9864b08e))
|
||||||
|
* **events:** migrate game-creation and bot flows to Redis Streams NCS-89 ([#62](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/62)) ([a24924c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/a24924c23057db3d700a75dbc4333557789cd991))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#46](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/46)) ([649566e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/649566eb3fcf38f91c8896a739f74ea318af312d))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#47](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/47)) ([87dfc6c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87dfc6c2bcce7f7d58fc641bd8d468a2e584c108))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* **official-bots:** NCS-70-auto-register official bots with account service ([#59](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/59)) ([7117a93](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/7117a93376272094d0b1a6abf2121254ce396684))
|
||||||
|
|
||||||
|
### Reverts
|
||||||
|
|
||||||
|
* Revert "refactor: update metrics paths formatting in application.yml for clarity" ([3870566](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/38705663498d5f47c40dafe2f26198589ede8656))
|
||||||
|
|||||||
@@ -77,6 +77,8 @@ dependencies {
|
|||||||
implementation(project(":modules:api"))
|
implementation(project(":modules:api"))
|
||||||
implementation(project(":modules:io"))
|
implementation(project(":modules:io"))
|
||||||
implementation(project(":modules:rule"))
|
implementation(project(":modules:rule"))
|
||||||
|
implementation(project(":modules:security"))
|
||||||
|
implementation("io.quarkus:quarkus-rest-client-jackson")
|
||||||
implementation("com.microsoft.onnxruntime:onnxruntime:${versions["ONNXRUNTIME"]!!}")
|
implementation("com.microsoft.onnxruntime:onnxruntime:${versions["ONNXRUNTIME"]!!}")
|
||||||
implementation("io.quarkus:quarkus-redis-client")
|
implementation("io.quarkus:quarkus-redis-client")
|
||||||
|
|
||||||
|
|||||||
@@ -5,6 +5,9 @@ quarkus:
|
|||||||
name: nowchess-official-bots
|
name: nowchess-official-bots
|
||||||
redis:
|
redis:
|
||||||
hosts: redis://${REDIS_HOST:localhost}:${REDIS_PORT:6379}
|
hosts: redis://${REDIS_HOST:localhost}:${REDIS_PORT:6379}
|
||||||
|
rest-client:
|
||||||
|
account-service:
|
||||||
|
url: http://localhost:8083
|
||||||
smallrye-jwt:
|
smallrye-jwt:
|
||||||
enabled: true
|
enabled: true
|
||||||
log:
|
log:
|
||||||
@@ -15,6 +18,8 @@ nowchess:
|
|||||||
host: localhost
|
host: localhost
|
||||||
port: 6379
|
port: 6379
|
||||||
prefix: nowchess
|
prefix: nowchess
|
||||||
|
internal:
|
||||||
|
secret: 123abc
|
||||||
|
|
||||||
"%deployed":
|
"%deployed":
|
||||||
quarkus:
|
quarkus:
|
||||||
@@ -28,8 +33,13 @@ nowchess:
|
|||||||
exporter:
|
exporter:
|
||||||
otlp:
|
otlp:
|
||||||
endpoint: ${OTEL_EXPORTER_OTLP_ENDPOINT:http://localhost:4317}
|
endpoint: ${OTEL_EXPORTER_OTLP_ENDPOINT:http://localhost:4317}
|
||||||
|
rest-client:
|
||||||
|
account-service:
|
||||||
|
url: ${ACCOUNT_SERVICE_URL}
|
||||||
nowchess:
|
nowchess:
|
||||||
redis:
|
redis:
|
||||||
host: ${REDIS_HOST:localhost}
|
host: ${REDIS_HOST:localhost}
|
||||||
port: ${REDIS_PORT:6379}
|
port: ${REDIS_PORT:6379}
|
||||||
prefix: ${REDIS_PREFIX:nowchess}
|
prefix: ${REDIS_PREFIX:nowchess}
|
||||||
|
internal:
|
||||||
|
secret: ${INTERNAL_SECRET}
|
||||||
|
|||||||
+20
@@ -0,0 +1,20 @@
|
|||||||
|
package de.nowchess.bot.client
|
||||||
|
|
||||||
|
import de.nowchess.security.{InternalClientHeadersFactory, InternalSecretClientFilter}
|
||||||
|
import jakarta.ws.rs.*
|
||||||
|
import jakarta.ws.rs.core.MediaType
|
||||||
|
import org.eclipse.microprofile.rest.client.annotation.{RegisterClientHeaders, RegisterProvider}
|
||||||
|
import org.eclipse.microprofile.rest.client.inject.RegisterRestClient
|
||||||
|
|
||||||
|
case class SyncOfficialBotsRequest(bots: List[String])
|
||||||
|
|
||||||
|
@Path("/api/account/official-bots")
|
||||||
|
@RegisterRestClient(configKey = "account-service")
|
||||||
|
@RegisterProvider(classOf[InternalSecretClientFilter])
|
||||||
|
@RegisterClientHeaders(classOf[InternalClientHeadersFactory])
|
||||||
|
trait AccountServiceClient:
|
||||||
|
|
||||||
|
@POST
|
||||||
|
@Path("/sync")
|
||||||
|
@Consumes(Array(MediaType.APPLICATION_JSON))
|
||||||
|
def syncBots(req: SyncOfficialBotsRequest): Unit
|
||||||
+114
-14
@@ -1,36 +1,61 @@
|
|||||||
package de.nowchess.bot.service
|
package de.nowchess.bot.service
|
||||||
|
|
||||||
import com.fasterxml.jackson.databind.ObjectMapper
|
import com.fasterxml.jackson.databind.ObjectMapper
|
||||||
|
import de.nowchess.api.event.EventEnvelope
|
||||||
import de.nowchess.api.move.{Move, MoveType, PromotionPiece}
|
import de.nowchess.api.move.{Move, MoveType, PromotionPiece}
|
||||||
import de.nowchess.bot.BotController
|
import de.nowchess.bot.BotController
|
||||||
import de.nowchess.bot.BotDifficulty
|
import de.nowchess.bot.BotDifficulty
|
||||||
|
import de.nowchess.bot.client.{AccountServiceClient, SyncOfficialBotsRequest}
|
||||||
import de.nowchess.bot.config.RedisConfig
|
import de.nowchess.bot.config.RedisConfig
|
||||||
import de.nowchess.io.fen.FenParser
|
import de.nowchess.io.fen.FenParser
|
||||||
import io.micrometer.core.instrument.MeterRegistry
|
import io.micrometer.core.instrument.MeterRegistry
|
||||||
import io.quarkus.redis.datasource.RedisDataSource
|
import io.quarkus.redis.datasource.RedisDataSource
|
||||||
|
import io.quarkus.redis.datasource.stream.{StreamMessage, XAddArgs, XGroupCreateArgs, XReadGroupArgs}
|
||||||
import io.quarkus.runtime.StartupEvent
|
import io.quarkus.runtime.StartupEvent
|
||||||
import jakarta.annotation.PostConstruct
|
import jakarta.annotation.PostConstruct
|
||||||
import jakarta.enterprise.context.ApplicationScoped
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
import jakarta.enterprise.event.Observes
|
import jakarta.enterprise.event.Observes
|
||||||
import jakarta.inject.Inject
|
import jakarta.inject.Inject
|
||||||
|
import org.eclipse.microprofile.context.ManagedExecutor
|
||||||
|
import org.eclipse.microprofile.rest.client.inject.RestClient
|
||||||
|
import org.jboss.logging.Logger
|
||||||
import scala.compiletime.uninitialized
|
import scala.compiletime.uninitialized
|
||||||
|
import scala.jdk.CollectionConverters.*
|
||||||
|
import scala.util.{Failure, Success, Try}
|
||||||
|
import java.time.Duration
|
||||||
|
import java.util.UUID
|
||||||
import java.util.function.Consumer
|
import java.util.function.Consumer
|
||||||
import java.util.concurrent.TimeUnit
|
import java.util.concurrent.TimeUnit
|
||||||
|
|
||||||
@ApplicationScoped
|
@ApplicationScoped
|
||||||
class OfficialBotService:
|
class OfficialBotService:
|
||||||
|
|
||||||
|
private val log = Logger.getLogger(classOf[OfficialBotService])
|
||||||
|
|
||||||
// scalafix:off DisableSyntax.var
|
// scalafix:off DisableSyntax.var
|
||||||
@Inject var redis: RedisDataSource = uninitialized
|
@Inject var redis: RedisDataSource = uninitialized
|
||||||
@Inject var redisConfig: RedisConfig = uninitialized
|
@Inject var redisConfig: RedisConfig = uninitialized
|
||||||
@Inject var objectMapper: ObjectMapper = uninitialized
|
@Inject var objectMapper: ObjectMapper = uninitialized
|
||||||
@Inject var botController: BotController = uninitialized
|
@Inject var botController: BotController = uninitialized
|
||||||
@Inject var meterRegistry: MeterRegistry = uninitialized
|
@Inject var meterRegistry: MeterRegistry = uninitialized
|
||||||
|
@Inject var executor: ManagedExecutor = uninitialized
|
||||||
|
|
||||||
|
@Inject
|
||||||
|
@RestClient
|
||||||
|
var accountServiceClient: AccountServiceClient = uninitialized
|
||||||
// scalafix:on DisableSyntax.var
|
// scalafix:on DisableSyntax.var
|
||||||
|
|
||||||
private val terminalStatuses =
|
private val terminalStatuses =
|
||||||
Set("checkmate", "resign", "timeout", "stalemate", "insufficientMaterial", "draw")
|
Set("checkmate", "resign", "timeout", "stalemate", "insufficientMaterial", "draw")
|
||||||
|
|
||||||
|
private val groupName = "official-bot"
|
||||||
|
private val consumerId = UUID.randomUUID().toString
|
||||||
|
private val maxRetries = 3
|
||||||
|
private val maxStreamLen = 1000L
|
||||||
|
|
||||||
|
private def eventStream(botName: String): String = s"${redisConfig.prefix}:bot:$botName:events:stream"
|
||||||
|
private def dlqStream: String = s"${redisConfig.prefix}:dlq"
|
||||||
|
|
||||||
@PostConstruct
|
@PostConstruct
|
||||||
def initializeMetrics(): Unit =
|
def initializeMetrics(): Unit =
|
||||||
BotController.listBots.foreach { bot =>
|
BotController.listBots.foreach { bot =>
|
||||||
@@ -39,23 +64,98 @@ class OfficialBotService:
|
|||||||
}
|
}
|
||||||
|
|
||||||
def onStart(@Observes event: StartupEvent): Unit =
|
def onStart(@Observes event: StartupEvent): Unit =
|
||||||
BotController.listBots.foreach(subscribeToEventChannel)
|
val bots = BotController.listBots
|
||||||
|
try accountServiceClient.syncBots(SyncOfficialBotsRequest(bots))
|
||||||
|
catch case ex: Exception => log.errorf(ex, "Failed to auto-register official bots with account service")
|
||||||
|
bots.foreach(subscribeToEventChannel)
|
||||||
|
|
||||||
private def subscribeToEventChannel(botName: String): Unit =
|
private def subscribeToEventChannel(botName: String): Unit =
|
||||||
val handler: Consumer[String] = msg => handleBotEvent(botName, msg)
|
createGroupIfAbsent(botName)
|
||||||
redis.pubsub(classOf[String]).subscribe(s"${redisConfig.prefix}:bot:$botName:events", handler)
|
executor.submit(
|
||||||
()
|
new Runnable:
|
||||||
|
def run(): Unit = pollLoop(botName),
|
||||||
|
)
|
||||||
|
log.infof("Listening to bot event stream for %s (consumer=%s)", botName, consumerId)
|
||||||
|
|
||||||
private def handleBotEvent(botName: String, msg: String): Unit =
|
private def createGroupIfAbsent(botName: String): Unit =
|
||||||
try
|
Try(
|
||||||
val node = objectMapper.readTree(msg)
|
redis
|
||||||
if node.path("type").asText() == "gameStart" then
|
.stream(classOf[String])
|
||||||
val gameId = node.path("gameId").asText()
|
.xgroupCreate(eventStream(botName), groupName, "0", new XGroupCreateArgs().mkstream()),
|
||||||
val playingAs = node.path("playingAs").asText()
|
) match
|
||||||
val difficulty = node.path("difficulty").asInt(1400)
|
case Failure(ex) if Option(ex.getMessage).exists(_.contains("BUSYGROUP")) => ()
|
||||||
val botAccountId = node.path("botAccountId").asText()
|
case Failure(ex) => log.warnf(ex, "Failed to create bot event consumer group for %s", botName)
|
||||||
watchGame(botName, gameId, playingAs, difficulty, botAccountId)
|
case Success(_) => ()
|
||||||
catch case _: Exception => ()
|
|
||||||
|
private def pollLoop(botName: String): Unit =
|
||||||
|
while true do
|
||||||
|
Try {
|
||||||
|
val messages = redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xreadgroup(
|
||||||
|
groupName,
|
||||||
|
consumerId,
|
||||||
|
eventStream(botName),
|
||||||
|
">",
|
||||||
|
new XReadGroupArgs().count(10).block(Duration.ofSeconds(2)),
|
||||||
|
)
|
||||||
|
Option(messages).foreach(_.forEach(msg => handleStreamMessage(botName, msg)))
|
||||||
|
} match
|
||||||
|
case Failure(ex) => log.warnf(ex, "Error in bot event poll loop for %s", botName)
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def handleStreamMessage(botName: String, msg: StreamMessage[String, String, String]): Unit =
|
||||||
|
val json = msg.payload().get("data")
|
||||||
|
val attempt = Option(msg.payload().get("attempt")).flatMap(_.toIntOption).getOrElse(0)
|
||||||
|
Try {
|
||||||
|
val envelope = objectMapper.readValue(json, classOf[EventEnvelope])
|
||||||
|
handleBotEvent(botName, envelope)
|
||||||
|
} match
|
||||||
|
case Success(_) => ack(botName, msg.id())
|
||||||
|
case Failure(ex) if attempt + 1 < maxRetries =>
|
||||||
|
log.warnf(ex, "Bot event handling failed for %s (attempt %d), retrying", botName, attempt)
|
||||||
|
retry(botName, json, attempt + 1)
|
||||||
|
ack(botName, msg.id())
|
||||||
|
case Failure(ex) =>
|
||||||
|
log.errorf(ex, "Bot event handling failed for %s after %d attempts, sending to DLQ", botName, maxRetries)
|
||||||
|
toDlq(json, ex, attempt)
|
||||||
|
ack(botName, msg.id())
|
||||||
|
|
||||||
|
private def handleBotEvent(botName: String, envelope: EventEnvelope): Unit =
|
||||||
|
val payload = envelope.payload
|
||||||
|
val gameId = payload.path("gameId").asText()
|
||||||
|
val playingAs = payload.path("playingAs").asText()
|
||||||
|
val difficulty = payload.path("difficulty").asInt(1400)
|
||||||
|
val botAccountId = payload.path("botAccountId").asText()
|
||||||
|
watchGame(botName, gameId, playingAs, difficulty, botAccountId)
|
||||||
|
|
||||||
|
private def ack(botName: String, id: String): Unit =
|
||||||
|
Try(redis.stream(classOf[String]).xack(eventStream(botName), groupName, id)) match
|
||||||
|
case Failure(ex) => log.warnf(ex, "Failed to ack bot event %s", id)
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
|
private def retry(botName: String, json: String, attempt: Int): Unit =
|
||||||
|
xadd(eventStream(botName), Map("data" -> json, "attempt" -> attempt.toString))
|
||||||
|
|
||||||
|
private def toDlq(json: String, error: Throwable, attempt: Int): Unit =
|
||||||
|
xadd(
|
||||||
|
dlqStream,
|
||||||
|
Map(
|
||||||
|
"data" -> json,
|
||||||
|
"eventType" -> "BotGameStart",
|
||||||
|
"error" -> Option(error.getMessage).getOrElse(error.getClass.getName),
|
||||||
|
"attempt" -> (attempt + 1).toString,
|
||||||
|
),
|
||||||
|
)
|
||||||
|
|
||||||
|
private def xadd(key: String, fields: Map[String, String]): Unit =
|
||||||
|
Try(
|
||||||
|
redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xadd(key, new XAddArgs().maxlen(maxStreamLen).nearlyExactTrimming(), fields.asJava),
|
||||||
|
) match
|
||||||
|
case Failure(ex) => log.errorf(ex, "Failed to publish to stream %s", key)
|
||||||
|
case Success(_) => ()
|
||||||
|
|
||||||
private def watchGame(
|
private def watchGame(
|
||||||
botName: String,
|
botName: String,
|
||||||
|
|||||||
@@ -1,3 +1,3 @@
|
|||||||
MAJOR=0
|
MAJOR=0
|
||||||
MINOR=13
|
MINOR=15
|
||||||
PATCH=0
|
PATCH=0
|
||||||
|
|||||||
@@ -453,3 +453,36 @@
|
|||||||
* **redis:** update Redis configuration with max pool size and waiting parameters ([5baf6a7](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5baf6a7cdbea484fc49c02e2b5a1c3919b7fa2c4))
|
* **redis:** update Redis configuration with max pool size and waiting parameters ([5baf6a7](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5baf6a7cdbea484fc49c02e2b5a1c3919b7fa2c4))
|
||||||
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
||||||
* **store:** replace null check with Option for stream messages ([252851d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/252851de1cd715f797847e0660ee501c3a77237e))
|
* **store:** replace null check with Option for stream messages ([252851d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/252851de1cd715f797847e0660ee501c3a77237e))
|
||||||
|
## (2026-06-03)
|
||||||
|
|
||||||
|
### Features
|
||||||
|
|
||||||
|
* add initialization metrics for various services ([d438e97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d438e97f32bdde0bfc63c1b4a8cc810cdd093166))
|
||||||
|
* add OpenTelemetry trace configuration with parentbased sampler ([3904d5a](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3904d5ad8ad4930ddee65287a7bfab785a6148f5))
|
||||||
|
* **config:** add GameWritebackEventDtoMixin for JSON deserialization ([381161f](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/381161f00345612a1789e08243746083dff884c5))
|
||||||
|
* **config:** update application.yml for PostgreSQL and remove staging/production configurations ([2404e61](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/2404e6164c3b50ffccbea5238d636060d6abe4d6))
|
||||||
|
* **config:** update application.yml for staging and production environments ([6113432](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/6113432a14c476a3a0dfc0d449e17d023697f2ba))
|
||||||
|
* **config:** update application.yml to nest HTTP port configuration ([3efebd5](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3efebd5ed0493159c51f7246d18d59bac58cf875))
|
||||||
|
* configure logging and add OpenTelemetry support ([#49](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/49)) ([d57c488](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d57c4886612d1d92da0e1b79209fc83e6ef537a1))
|
||||||
|
* **docker:** add .dockerignore and .gitignore files for build exclusions ([c987d8e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/c987d8e258c0e6c4cfbdaa8381c64c410d7a2b83))
|
||||||
|
* **docker:** add Dockerfiles for building Quarkus application in native and JVM modes ([3f2d2bb](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3f2d2bb4c97fa8cddba66e1da4427c54236dfeed))
|
||||||
|
* **docker:** add Dockerfiles for Quarkus application in JVM and native modes ([34b9933](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/34b993304670cf2aa62cd2f6460cee7b9864b08e))
|
||||||
|
* **dto:** update GameWritebackEventDto for JSON deserialization and remove unused mixin ([576e3fe](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/576e3fea9bf1082549ea53efd3288474c42be93d))
|
||||||
|
* implement clock expiry scanning and handling for game records ([#53](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/53)) ([8f9eb12](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/8f9eb12f663efabe4dc72b94394438652ad0ef02))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#46](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/46)) ([649566e](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/649566eb3fcf38f91c8896a739f74ea318af312d))
|
||||||
|
* NCS-78 Add Traceability to the Applications ([#47](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/47)) ([87dfc6c](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/87dfc6c2bcce7f7d58fc641bd8d468a2e584c108))
|
||||||
|
* **redis:** add @Startup annotation to GameWritebackStreamListener ([d61fe97](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/d61fe97b4c8e2db5e34b4a14d995297cc09f9435))
|
||||||
|
* **redis:** implement game writeback stream processing with error handling and retries ([ae3ef76](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/ae3ef766e8b7596a09e466cd4fb386119f17ca5c))
|
||||||
|
* **redis:** use ManagedExecutor for asynchronous writeback processing ([af6b0ed](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/af6b0ed8b73724fcc8f20dfccbe6fe8f84fd792d))
|
||||||
|
* true-microservices ([#40](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/40)) ([5909242](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/590924254e8a2754de661a57a03e43f89ceb6299))
|
||||||
|
* update application.yml with new API root paths and add Micrometer and OpenTelemetry dependencies ([72ce262](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/72ce262bc491f94297700e6002fb5d0812e2cc2a))
|
||||||
|
|
||||||
|
### Bug Fixes
|
||||||
|
|
||||||
|
* ensure full hierarchy registration for reflection in NativeReflectionConfig ([ebba729](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/ebba729af3265df1619dfbe46fd1945b2a7e30b7))
|
||||||
|
* NCS-85 Database Writeback fails without Logs ([#52](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/52)) ([7323908](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/73239088d985f01aa6b1067ed9097a845e471d4f))
|
||||||
|
* **redis:** add log message for starting Writeback listener ([b610678](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/b610678005de645115f48348e66aa9e6f5deb3d5))
|
||||||
|
* **redis:** update Redis configuration with max pool size and waiting parameters ([5baf6a7](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/5baf6a7cdbea484fc49c02e2b5a1c3919b7fa2c4))
|
||||||
|
* remove unused HTTP root-path configurations from application.yml ([3ed3e59](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/3ed3e59ee456d54cd3d65ece4f36623e256b9736))
|
||||||
|
* **store:** cap game-writeback stream with MAXLEN trimming ([#58](https://git.janis-eccarius.de/NowChess/NowChessSystems/issues/58)) ([32c3887](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/32c388760b4f40f2937e4f3b4a04495313144fc0))
|
||||||
|
* **store:** replace null check with Option for stream messages ([252851d](https://git.janis-eccarius.de/NowChess/NowChessSystems/commit/252851de1cd715f797847e0660ee501c3a77237e))
|
||||||
|
|||||||
+15
-6
@@ -5,7 +5,7 @@ import de.nowchess.api.dto.GameWritebackEventDto
|
|||||||
import de.nowchess.store.config.RedisConfig
|
import de.nowchess.store.config.RedisConfig
|
||||||
import de.nowchess.store.service.GameWritebackService
|
import de.nowchess.store.service.GameWritebackService
|
||||||
import io.quarkus.redis.datasource.RedisDataSource
|
import io.quarkus.redis.datasource.RedisDataSource
|
||||||
import io.quarkus.redis.datasource.stream.{StreamMessage, XGroupCreateArgs, XReadGroupArgs}
|
import io.quarkus.redis.datasource.stream.{StreamMessage, XAddArgs, XGroupCreateArgs, XReadGroupArgs}
|
||||||
import io.quarkus.runtime.Startup
|
import io.quarkus.runtime.Startup
|
||||||
import jakarta.annotation.PostConstruct
|
import jakarta.annotation.PostConstruct
|
||||||
import jakarta.enterprise.context.ApplicationScoped
|
import jakarta.enterprise.context.ApplicationScoped
|
||||||
@@ -32,10 +32,11 @@ class GameWritebackStreamListener:
|
|||||||
private val log = Logger.getLogger(classOf[GameWritebackStreamListener])
|
private val log = Logger.getLogger(classOf[GameWritebackStreamListener])
|
||||||
private val groupName = "store-writeback"
|
private val groupName = "store-writeback"
|
||||||
|
|
||||||
private def streamKey = s"${redisConfig.prefix}:game-writeback"
|
private def streamKey = s"${redisConfig.prefix}:game-writeback"
|
||||||
private def dlqKey = s"${redisConfig.prefix}:game-writeback-dlq"
|
private def dlqKey = s"${redisConfig.prefix}:game-writeback-dlq"
|
||||||
private val maxRetries = 3
|
private val maxRetries = 3
|
||||||
private val consumerId = UUID.randomUUID().toString
|
private val consumerId = UUID.randomUUID().toString
|
||||||
|
private val maxStreamLen = 1000L
|
||||||
|
|
||||||
@PostConstruct
|
@PostConstruct
|
||||||
def startListening(): Unit =
|
def startListening(): Unit =
|
||||||
@@ -98,6 +99,14 @@ class GameWritebackStreamListener:
|
|||||||
case Success(_) => ()
|
case Success(_) => ()
|
||||||
|
|
||||||
private def xadd(key: String, json: String, attempt: Int): Unit =
|
private def xadd(key: String, json: String, attempt: Int): Unit =
|
||||||
Try(redis.stream(classOf[String]).xadd(key, Map("data" -> json, "attempt" -> attempt.toString).asJava)) match
|
Try(
|
||||||
|
redis
|
||||||
|
.stream(classOf[String])
|
||||||
|
.xadd(
|
||||||
|
key,
|
||||||
|
new XAddArgs().maxlen(maxStreamLen).nearlyExactTrimming(),
|
||||||
|
Map("data" -> json, "attempt" -> attempt.toString).asJava,
|
||||||
|
),
|
||||||
|
) match
|
||||||
case Failure(ex) => log.errorf(ex, "Failed to publish to stream %s", key)
|
case Failure(ex) => log.errorf(ex, "Failed to publish to stream %s", key)
|
||||||
case Success(_) => ()
|
case Success(_) => ()
|
||||||
|
|||||||
@@ -1,3 +1,3 @@
|
|||||||
MAJOR=0
|
MAJOR=0
|
||||||
MINOR=23
|
MINOR=24
|
||||||
PATCH=0
|
PATCH=0
|
||||||
|
|||||||
Reference in New Issue
Block a user