feat(phase1): 完成 Repository 测试和 Redis 会话管理
Task 2: 完成 Repository 层测试 - 实现 CaseLibraryRepositoryTest (6 个测试) - 实现 ApiDocumentRepositoryTest (7 个测试) - 所有 Repository 测试通过 (19/19) Task 3: 完成 Redis 会话管理 - 创建 SessionContext 和 ToolCall 数据类 - 创建 SessionManager 接口 - 实现 RedisSessionManager (基于 Redis 的会话管理) - 创建 SessionConfiguration (Redis 序列化配置) - 实现 RedisSessionManagerTest (8 个测试全部通过) 测试结果： - ApiDocumentRepositoryTest: 7/7 通过 - CaseLibraryRepositoryTest: 6/6 通过 - DiagnosisRecordRepositoryTest: 6/6 通过 - RedisSessionManagerTest: 8/8 通过 - 总计: 27/27 测试通过 功能特性： - 会话创建、查询、更新、删除 - 会话过期时间管理 - 工具调用记录追踪 - 会话状态管理 - 基于 Redis 的分布式会话存储 Progress: 20/33 tasks completed (61%)
This commit is contained in:
zhuyongxin
@@ -1,152 +0,0 @@
|
|||||||
---
|
|
||||||
name: "OPSX: Apply"
|
|
||||||
description: Implement tasks from an OpenSpec change (Experimental)
|
|
||||||
category: Workflow
|
|
||||||
tags: [workflow, artifacts, experimental]
|
|
||||||
---
|
|
||||||
|
|
||||||
Implement tasks from an OpenSpec change.
|
|
||||||
|
|
||||||
**Input**: Optionally specify a change name (e.g., `/opsx:apply add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
|
|
||||||
|
|
||||||
**Steps**
|
|
||||||
|
|
||||||
1. **Select the change**
|
|
||||||
|
|
||||||
If a name is provided, use it. Otherwise:
|
|
||||||
- Infer from conversation context if the user mentioned a change
|
|
||||||
- Auto-select if only one active change exists
|
|
||||||
- If ambiguous, run `openspec list --json` to get available changes and use the **AskUserQuestion tool** to let the user select
|
|
||||||
|
|
||||||
Always announce: "Using change: <name>" and how to override (e.g., `/opsx:apply <other>`).
|
|
||||||
|
|
||||||
2. **Check status to understand the schema**
|
|
||||||
```bash
|
|
||||||
openspec status --change "<name>" --json
|
|
||||||
```
|
|
||||||
Parse the JSON to understand:
|
|
||||||
- `schemaName`: The workflow being used (e.g., "spec-driven")
|
|
||||||
- Which artifact contains the tasks (typically "tasks" for spec-driven, check status for others)
|
|
||||||
|
|
||||||
3. **Get apply instructions**
|
|
||||||
|
|
||||||
```bash
|
|
||||||
openspec instructions apply --change "<name>" --json
|
|
||||||
```
|
|
||||||
|
|
||||||
This returns:
|
|
||||||
- `contextFiles`: artifact ID -> array of concrete file paths (varies by schema)
|
|
||||||
- Progress (total, complete, remaining)
|
|
||||||
- Task list with status
|
|
||||||
- Dynamic instruction based on current state
|
|
||||||
|
|
||||||
**Handle states:**
|
|
||||||
- If `state: "blocked"` (missing artifacts): show message, suggest using `/opsx:continue`
|
|
||||||
- If `state: "all_done"`: congratulate, suggest archive
|
|
||||||
- Otherwise: proceed to implementation
|
|
||||||
|
|
||||||
4. **Read context files**
|
|
||||||
|
|
||||||
Read every file path listed under `contextFiles` from the apply instructions output.
|
|
||||||
The files depend on the schema being used:
|
|
||||||
- **spec-driven**: proposal, specs, design, tasks
|
|
||||||
- Other schemas: follow the contextFiles from CLI output
|
|
||||||
|
|
||||||
5. **Show current progress**
|
|
||||||
|
|
||||||
Display:
|
|
||||||
- Schema being used
|
|
||||||
- Progress: "N/M tasks complete"
|
|
||||||
- Remaining tasks overview
|
|
||||||
- Dynamic instruction from CLI
|
|
||||||
|
|
||||||
6. **Implement tasks (loop until done or blocked)**
|
|
||||||
|
|
||||||
For each pending task:
|
|
||||||
- Show which task is being worked on
|
|
||||||
- Make the code changes required
|
|
||||||
- Keep changes minimal and focused
|
|
||||||
- Mark task complete in the tasks file: `- [ ]` → `- [x]`
|
|
||||||
- Continue to next task
|
|
||||||
|
|
||||||
**Pause if:**
|
|
||||||
- Task is unclear → ask for clarification
|
|
||||||
- Implementation reveals a design issue → suggest updating artifacts
|
|
||||||
- Error or blocker encountered → report and wait for guidance
|
|
||||||
- User interrupts
|
|
||||||
|
|
||||||
7. **On completion or pause, show status**
|
|
||||||
|
|
||||||
Display:
|
|
||||||
- Tasks completed this session
|
|
||||||
- Overall progress: "N/M tasks complete"
|
|
||||||
- If all done: suggest archive
|
|
||||||
- If paused: explain why and wait for guidance
|
|
||||||
|
|
||||||
**Output During Implementation**
|
|
||||||
|
|
||||||
```
|
|
||||||
## Implementing: <change-name> (schema: <schema-name>)
|
|
||||||
|
|
||||||
Working on task 3/7: <task description>
|
|
||||||
[...implementation happening...]
|
|
||||||
✓ Task complete
|
|
||||||
|
|
||||||
Working on task 4/7: <task description>
|
|
||||||
[...implementation happening...]
|
|
||||||
✓ Task complete
|
|
||||||
```
|
|
||||||
|
|
||||||
**Output On Completion**
|
|
||||||
|
|
||||||
```
|
|
||||||
## Implementation Complete
|
|
||||||
|
|
||||||
**Change:** <change-name>
|
|
||||||
**Schema:** <schema-name>
|
|
||||||
**Progress:** 7/7 tasks complete ✓
|
|
||||||
|
|
||||||
### Completed This Session
|
|
||||||
- [x] Task 1
|
|
||||||
- [x] Task 2
|
|
||||||
...
|
|
||||||
|
|
||||||
All tasks complete! You can archive this change with `/opsx:archive`.
|
|
||||||
```
|
|
||||||
|
|
||||||
**Output On Pause (Issue Encountered)**
|
|
||||||
|
|
||||||
```
|
|
||||||
## Implementation Paused
|
|
||||||
|
|
||||||
**Change:** <change-name>
|
|
||||||
**Schema:** <schema-name>
|
|
||||||
**Progress:** 4/7 tasks complete
|
|
||||||
|
|
||||||
### Issue Encountered
|
|
||||||
<description of the issue>
|
|
||||||
|
|
||||||
**Options:**
|
|
||||||
1. <option 1>
|
|
||||||
2. <option 2>
|
|
||||||
3. Other approach
|
|
||||||
|
|
||||||
What would you like to do?
|
|
||||||
```
|
|
||||||
|
|
||||||
**Guardrails**
|
|
||||||
- Keep going through tasks until done or blocked
|
|
||||||
- Always read context files before starting (from the apply instructions output)
|
|
||||||
- If task is ambiguous, pause and ask before implementing
|
|
||||||
- If implementation reveals issues, pause and suggest artifact updates
|
|
||||||
- Keep code changes minimal and scoped to each task
|
|
||||||
- Update task checkbox immediately after completing each task
|
|
||||||
- Pause on errors, blockers, or unclear requirements - don't guess
|
|
||||||
- Use contextFiles from CLI output, don't assume specific file names
|
|
||||||
|
|
||||||
**Fluid Workflow Integration**
|
|
||||||
|
|
||||||
This skill supports the "actions on a change" model:
|
|
||||||
|
|
||||||
- **Can be invoked anytime**: Before all artifacts are done (if tasks exist), after partial implementation, interleaved with other actions
|
|
||||||
- **Allows artifact updates**: If implementation reveals design issues, suggest updating artifacts - not phase-locked, work fluidly
|
|
||||||
@@ -1,157 +0,0 @@
|
|||||||
---
|
|
||||||
name: "OPSX: Archive"
|
|
||||||
description: Archive a completed change in the experimental workflow
|
|
||||||
category: Workflow
|
|
||||||
tags: [workflow, archive, experimental]
|
|
||||||
---
|
|
||||||
|
|
||||||
Archive a completed change in the experimental workflow.
|
|
||||||
|
|
||||||
**Input**: Optionally specify a change name after `/opsx:archive` (e.g., `/opsx:archive add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
|
|
||||||
|
|
||||||
**Steps**
|
|
||||||
|
|
||||||
1. **If no change name provided, prompt for selection**
|
|
||||||
|
|
||||||
Run `openspec list --json` to get available changes. Use the **AskUserQuestion tool** to let the user select.
|
|
||||||
|
|
||||||
Show only active changes (not already archived).
|
|
||||||
Include the schema used for each change if available.
|
|
||||||
|
|
||||||
**IMPORTANT**: Do NOT guess or auto-select a change. Always let the user choose.
|
|
||||||
|
|
||||||
2. **Check artifact completion status**
|
|
||||||
|
|
||||||
Run `openspec status --change "<name>" --json` to check artifact completion.
|
|
||||||
|
|
||||||
Parse the JSON to understand:
|
|
||||||
- `schemaName`: The workflow being used
|
|
||||||
- `artifacts`: List of artifacts with their status (`done` or other)
|
|
||||||
|
|
||||||
**If any artifacts are not `done`:**
|
|
||||||
- Display warning listing incomplete artifacts
|
|
||||||
- Prompt user for confirmation to continue
|
|
||||||
- Proceed if user confirms
|
|
||||||
|
|
||||||
3. **Check task completion status**
|
|
||||||
|
|
||||||
Read the tasks file (typically `tasks.md`) to check for incomplete tasks.
|
|
||||||
|
|
||||||
Count tasks marked with `- [ ]` (incomplete) vs `- [x]` (complete).
|
|
||||||
|
|
||||||
**If incomplete tasks found:**
|
|
||||||
- Display warning showing count of incomplete tasks
|
|
||||||
- Prompt user for confirmation to continue
|
|
||||||
- Proceed if user confirms
|
|
||||||
|
|
||||||
**If no tasks file exists:** Proceed without task-related warning.
|
|
||||||
|
|
||||||
4. **Assess delta spec sync state**
|
|
||||||
|
|
||||||
Check for delta specs at `openspec/changes/<name>/specs/`. If none exist, proceed without sync prompt.
|
|
||||||
|
|
||||||
**If delta specs exist:**
|
|
||||||
- Compare each delta spec with its corresponding main spec at `openspec/specs/<capability>/spec.md`
|
|
||||||
- Determine what changes would be applied (adds, modifications, removals, renames)
|
|
||||||
- Show a combined summary before prompting
|
|
||||||
|
|
||||||
**Prompt options:**
|
|
||||||
- If changes needed: "Sync now (recommended)", "Archive without syncing"
|
|
||||||
- If already synced: "Archive now", "Sync anyway", "Cancel"
|
|
||||||
|
|
||||||
If user chooses sync, use Task tool (subagent_type: "general-purpose", prompt: "Use Skill tool to invoke openspec-sync-specs for change '<name>'. Delta spec analysis: <include the analyzed delta spec summary>"). Proceed to archive regardless of choice.
|
|
||||||
|
|
||||||
5. **Perform the archive**
|
|
||||||
|
|
||||||
Create the archive directory if it doesn't exist:
|
|
||||||
```bash
|
|
||||||
mkdir -p openspec/changes/archive
|
|
||||||
```
|
|
||||||
|
|
||||||
Generate target name using current date: `YYYY-MM-DD-<change-name>`
|
|
||||||
|
|
||||||
**Check if target already exists:**
|
|
||||||
- If yes: Fail with error, suggest renaming existing archive or using different date
|
|
||||||
- If no: Move the change directory to archive
|
|
||||||
|
|
||||||
```bash
|
|
||||||
mv openspec/changes/<name> openspec/changes/archive/YYYY-MM-DD-<name>
|
|
||||||
```
|
|
||||||
|
|
||||||
6. **Display summary**
|
|
||||||
|
|
||||||
Show archive completion summary including:
|
|
||||||
- Change name
|
|
||||||
- Schema that was used
|
|
||||||
- Archive location
|
|
||||||
- Spec sync status (synced / sync skipped / no delta specs)
|
|
||||||
- Note about any warnings (incomplete artifacts/tasks)
|
|
||||||
|
|
||||||
**Output On Success**
|
|
||||||
|
|
||||||
```
|
|
||||||
## Archive Complete
|
|
||||||
|
|
||||||
**Change:** <change-name>
|
|
||||||
**Schema:** <schema-name>
|
|
||||||
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
|
|
||||||
**Specs:** ✓ Synced to main specs
|
|
||||||
|
|
||||||
All artifacts complete. All tasks complete.
|
|
||||||
```
|
|
||||||
|
|
||||||
**Output On Success (No Delta Specs)**
|
|
||||||
|
|
||||||
```
|
|
||||||
## Archive Complete
|
|
||||||
|
|
||||||
**Change:** <change-name>
|
|
||||||
**Schema:** <schema-name>
|
|
||||||
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
|
|
||||||
**Specs:** No delta specs
|
|
||||||
|
|
||||||
All artifacts complete. All tasks complete.
|
|
||||||
```
|
|
||||||
|
|
||||||
**Output On Success With Warnings**
|
|
||||||
|
|
||||||
```
|
|
||||||
## Archive Complete (with warnings)
|
|
||||||
|
|
||||||
**Change:** <change-name>
|
|
||||||
**Schema:** <schema-name>
|
|
||||||
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
|
|
||||||
**Specs:** Sync skipped (user chose to skip)
|
|
||||||
|
|
||||||
**Warnings:**
|
|
||||||
- Archived with 2 incomplete artifacts
|
|
||||||
- Archived with 3 incomplete tasks
|
|
||||||
- Delta spec sync was skipped (user chose to skip)
|
|
||||||
|
|
||||||
Review the archive if this was not intentional.
|
|
||||||
```
|
|
||||||
|
|
||||||
**Output On Error (Archive Exists)**
|
|
||||||
|
|
||||||
```
|
|
||||||
## Archive Failed
|
|
||||||
|
|
||||||
**Change:** <change-name>
|
|
||||||
**Target:** openspec/changes/archive/YYYY-MM-DD-<name>/
|
|
||||||
|
|
||||||
Target archive directory already exists.
|
|
||||||
|
|
||||||
**Options:**
|
|
||||||
1. Rename the existing archive
|
|
||||||
2. Delete the existing archive if it's a duplicate
|
|
||||||
3. Wait until a different date to archive
|
|
||||||
```
|
|
||||||
|
|
||||||
**Guardrails**
|
|
||||||
- Always prompt for change selection if not provided
|
|
||||||
- Use artifact graph (openspec status --json) for completion checking
|
|
||||||
- Don't block archive on warnings - just inform and confirm
|
|
||||||
- Preserve .openspec.yaml when moving to archive (it moves with the directory)
|
|
||||||
- Show clear summary of what happened
|
|
||||||
- If sync is requested, use the Skill tool to invoke `openspec-sync-specs` (agent-driven)
|
|
||||||
- If delta specs exist, always run the sync assessment and show the combined summary before prompting
|
|
||||||
@@ -1,173 +0,0 @@
|
|||||||
---
|
|
||||||
name: "OPSX: Explore"
|
|
||||||
description: "Enter explore mode - think through ideas, investigate problems, clarify requirements"
|
|
||||||
category: Workflow
|
|
||||||
tags: [workflow, explore, experimental, thinking]
|
|
||||||
---
|
|
||||||
|
|
||||||
Enter explore mode. Think deeply. Visualize freely. Follow the conversation wherever it goes.
|
|
||||||
|
|
||||||
**IMPORTANT: Explore mode is for thinking, not implementing.** You may read files, search code, and investigate the codebase, but you must NEVER write code or implement features. If the user asks you to implement something, remind them to exit explore mode first and create a change proposal. You MAY create OpenSpec artifacts (proposals, designs, specs) if the user asks—that's capturing thinking, not implementing.
|
|
||||||
|
|
||||||
**This is a stance, not a workflow.** There are no fixed steps, no required sequence, no mandatory outputs. You're a thinking partner helping the user explore.
|
|
||||||
|
|
||||||
**Input**: The argument after `/opsx:explore` is whatever the user wants to think about. Could be:
|
|
||||||
- A vague idea: "real-time collaboration"
|
|
||||||
- A specific problem: "the auth system is getting unwieldy"
|
|
||||||
- A change name: "add-dark-mode" (to explore in context of that change)
|
|
||||||
- A comparison: "postgres vs sqlite for this"
|
|
||||||
- Nothing (just enter explore mode)
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
## The Stance
|
|
||||||
|
|
||||||
- **Curious, not prescriptive** - Ask questions that emerge naturally, don't follow a script
|
|
||||||
- **Open threads, not interrogations** - Surface multiple interesting directions and let the user follow what resonates. Don't funnel them through a single path of questions.
|
|
||||||
- **Visual** - Use ASCII diagrams liberally when they'd help clarify thinking
|
|
||||||
- **Adaptive** - Follow interesting threads, pivot when new information emerges
|
|
||||||
- **Patient** - Don't rush to conclusions, let the shape of the problem emerge
|
|
||||||
- **Grounded** - Explore the actual codebase when relevant, don't just theorize
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
## What You Might Do
|
|
||||||
|
|
||||||
Depending on what the user brings, you might:
|
|
||||||
|
|
||||||
**Explore the problem space**
|
|
||||||
- Ask clarifying questions that emerge from what they said
|
|
||||||
- Challenge assumptions
|
|
||||||
- Reframe the problem
|
|
||||||
- Find analogies
|
|
||||||
|
|
||||||
**Investigate the codebase**
|
|
||||||
- Map existing architecture relevant to the discussion
|
|
||||||
- Find integration points
|
|
||||||
- Identify patterns already in use
|
|
||||||
- Surface hidden complexity
|
|
||||||
|
|
||||||
**Compare options**
|
|
||||||
- Brainstorm multiple approaches
|
|
||||||
- Build comparison tables
|
|
||||||
- Sketch tradeoffs
|
|
||||||
- Recommend a path (if asked)
|
|
||||||
|
|
||||||
**Visualize**
|
|
||||||
```
|
|
||||||
┌─────────────────────────────────────────┐
|
|
||||||
│ Use ASCII diagrams liberally │
|
|
||||||
├─────────────────────────────────────────┤
|
|
||||||
│ │
|
|
||||||
│ ┌────────┐ ┌────────┐ │
|
|
||||||
│ │ State │────────▶│ State │ │
|
|
||||||
│ │ A │ │ B │ │
|
|
||||||
│ └────────┘ └────────┘ │
|
|
||||||
│ │
|
|
||||||
│ System diagrams, state machines, │
|
|
||||||
│ data flows, architecture sketches, │
|
|
||||||
│ dependency graphs, comparison tables │
|
|
||||||
│ │
|
|
||||||
└─────────────────────────────────────────┘
|
|
||||||
```
|
|
||||||
|
|
||||||
**Surface risks and unknowns**
|
|
||||||
- Identify what could go wrong
|
|
||||||
- Find gaps in understanding
|
|
||||||
- Suggest spikes or investigations
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
## OpenSpec Awareness
|
|
||||||
|
|
||||||
You have full context of the OpenSpec system. Use it naturally, don't force it.
|
|
||||||
|
|
||||||
### Check for context
|
|
||||||
|
|
||||||
At the start, quickly check what exists:
|
|
||||||
```bash
|
|
||||||
openspec list --json
|
|
||||||
```
|
|
||||||
|
|
||||||
This tells you:
|
|
||||||
- If there are active changes
|
|
||||||
- Their names, schemas, and status
|
|
||||||
- What the user might be working on
|
|
||||||
|
|
||||||
If the user mentioned a specific change name, read its artifacts for context.
|
|
||||||
|
|
||||||
### When no change exists
|
|
||||||
|
|
||||||
Think freely. When insights crystallize, you might offer:
|
|
||||||
|
|
||||||
- "This feels solid enough to start a change. Want me to create a proposal?"
|
|
||||||
- Or keep exploring - no pressure to formalize
|
|
||||||
|
|
||||||
### When a change exists
|
|
||||||
|
|
||||||
If the user mentions a change or you detect one is relevant:
|
|
||||||
|
|
||||||
1. **Read existing artifacts for context**
|
|
||||||
- `openspec/changes/<name>/proposal.md`
|
|
||||||
- `openspec/changes/<name>/design.md`
|
|
||||||
- `openspec/changes/<name>/tasks.md`
|
|
||||||
- etc.
|
|
||||||
|
|
||||||
2. **Reference them naturally in conversation**
|
|
||||||
- "Your design mentions using Redis, but we just realized SQLite fits better..."
|
|
||||||
- "The proposal scopes this to premium users, but we're now thinking everyone..."
|
|
||||||
|
|
||||||
3. **Offer to capture when decisions are made**
|
|
||||||
|
|
||||||
| Insight Type | Where to Capture |
|
|
||||||
|----------------------------|--------------------------------|
|
|
||||||
| New requirement discovered | `specs/<capability>/spec.md` |
|
|
||||||
| Requirement changed | `specs/<capability>/spec.md` |
|
|
||||||
| Design decision made | `design.md` |
|
|
||||||
| Scope changed | `proposal.md` |
|
|
||||||
| New work identified | `tasks.md` |
|
|
||||||
| Assumption invalidated | Relevant artifact |
|
|
||||||
|
|
||||||
Example offers:
|
|
||||||
- "That's a design decision. Capture it in design.md?"
|
|
||||||
- "This is a new requirement. Add it to specs?"
|
|
||||||
- "This changes scope. Update the proposal?"
|
|
||||||
|
|
||||||
4. **The user decides** - Offer and move on. Don't pressure. Don't auto-capture.
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
## What You Don't Have To Do
|
|
||||||
|
|
||||||
- Follow a script
|
|
||||||
- Ask the same questions every time
|
|
||||||
- Produce a specific artifact
|
|
||||||
- Reach a conclusion
|
|
||||||
- Stay on topic if a tangent is valuable
|
|
||||||
- Be brief (this is thinking time)
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
## Ending Discovery
|
|
||||||
|
|
||||||
There's no required ending. Discovery might:
|
|
||||||
|
|
||||||
- **Flow into a proposal**: "Ready to start? I can create a change proposal."
|
|
||||||
- **Result in artifact updates**: "Updated design.md with these decisions"
|
|
||||||
- **Just provide clarity**: User has what they need, moves on
|
|
||||||
- **Continue later**: "We can pick this up anytime"
|
|
||||||
|
|
||||||
When things crystallize, you might offer a summary - but it's optional. Sometimes the thinking IS the value.
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
## Guardrails
|
|
||||||
|
|
||||||
- **Don't implement** - Never write code or implement features. Creating OpenSpec artifacts is fine, writing application code is not.
|
|
||||||
- **Don't fake understanding** - If something is unclear, dig deeper
|
|
||||||
- **Don't rush** - Discovery is thinking time, not task time
|
|
||||||
- **Don't force structure** - Let patterns emerge naturally
|
|
||||||
- **Don't auto-capture** - Offer to save insights, don't just do it
|
|
||||||
- **Do visualize** - A good diagram is worth many paragraphs
|
|
||||||
- **Do explore the codebase** - Ground discussions in reality
|
|
||||||
- **Do question assumptions** - Including the user's and your own
|
|
||||||
@@ -1,106 +0,0 @@
|
|||||||
---
|
|
||||||
name: "OPSX: Propose"
|
|
||||||
description: Propose a new change - create it and generate all artifacts in one step
|
|
||||||
category: Workflow
|
|
||||||
tags: [workflow, artifacts, experimental]
|
|
||||||
---
|
|
||||||
|
|
||||||
Propose a new change - create the change and generate all artifacts in one step.
|
|
||||||
|
|
||||||
I'll create a change with artifacts:
|
|
||||||
- proposal.md (what & why)
|
|
||||||
- design.md (how)
|
|
||||||
- tasks.md (implementation steps)
|
|
||||||
|
|
||||||
When ready to implement, run /opsx:apply
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
**Input**: The argument after `/opsx:propose` is the change name (kebab-case), OR a description of what the user wants to build.
|
|
||||||
|
|
||||||
**Steps**
|
|
||||||
|
|
||||||
1. **If no input provided, ask what they want to build**
|
|
||||||
|
|
||||||
Use the **AskUserQuestion tool** (open-ended, no preset options) to ask:
|
|
||||||
> "What change do you want to work on? Describe what you want to build or fix."
|
|
||||||
|
|
||||||
From their description, derive a kebab-case name (e.g., "add user authentication" → `add-user-auth`).
|
|
||||||
|
|
||||||
**IMPORTANT**: Do NOT proceed without understanding what the user wants to build.
|
|
||||||
|
|
||||||
2. **Create the change directory**
|
|
||||||
```bash
|
|
||||||
openspec new change "<name>"
|
|
||||||
```
|
|
||||||
This creates a scaffolded change at `openspec/changes/<name>/` with `.openspec.yaml`.
|
|
||||||
|
|
||||||
3. **Get the artifact build order**
|
|
||||||
```bash
|
|
||||||
openspec status --change "<name>" --json
|
|
||||||
```
|
|
||||||
Parse the JSON to get:
|
|
||||||
- `applyRequires`: array of artifact IDs needed before implementation (e.g., `["tasks"]`)
|
|
||||||
- `artifacts`: list of all artifacts with their status and dependencies
|
|
||||||
|
|
||||||
4. **Create artifacts in sequence until apply-ready**
|
|
||||||
|
|
||||||
Use the **TodoWrite tool** to track progress through the artifacts.
|
|
||||||
|
|
||||||
Loop through artifacts in dependency order (artifacts with no pending dependencies first):
|
|
||||||
|
|
||||||
a. **For each artifact that is `ready` (dependencies satisfied)**:
|
|
||||||
- Get instructions:
|
|
||||||
```bash
|
|
||||||
openspec instructions <artifact-id> --change "<name>" --json
|
|
||||||
```
|
|
||||||
- The instructions JSON includes:
|
|
||||||
- `context`: Project background (constraints for you - do NOT include in output)
|
|
||||||
- `rules`: Artifact-specific rules (constraints for you - do NOT include in output)
|
|
||||||
- `template`: The structure to use for your output file
|
|
||||||
- `instruction`: Schema-specific guidance for this artifact type
|
|
||||||
- `outputPath`: Where to write the artifact
|
|
||||||
- `dependencies`: Completed artifacts to read for context
|
|
||||||
- Read any completed dependency files for context
|
|
||||||
- Create the artifact file using `template` as the structure
|
|
||||||
- Apply `context` and `rules` as constraints - but do NOT copy them into the file
|
|
||||||
- Show brief progress: "Created <artifact-id>"
|
|
||||||
|
|
||||||
b. **Continue until all `applyRequires` artifacts are complete**
|
|
||||||
- After creating each artifact, re-run `openspec status --change "<name>" --json`
|
|
||||||
- Check if every artifact ID in `applyRequires` has `status: "done"` in the artifacts array
|
|
||||||
- Stop when all `applyRequires` artifacts are done
|
|
||||||
|
|
||||||
c. **If an artifact requires user input** (unclear context):
|
|
||||||
- Use **AskUserQuestion tool** to clarify
|
|
||||||
- Then continue with creation
|
|
||||||
|
|
||||||
5. **Show final status**
|
|
||||||
```bash
|
|
||||||
openspec status --change "<name>"
|
|
||||||
```
|
|
||||||
|
|
||||||
**Output**
|
|
||||||
|
|
||||||
After completing all artifacts, summarize:
|
|
||||||
- Change name and location
|
|
||||||
- List of artifacts created with brief descriptions
|
|
||||||
- What's ready: "All artifacts created! Ready for implementation."
|
|
||||||
- Prompt: "Run `/opsx:apply` to start implementing."
|
|
||||||
|
|
||||||
**Artifact Creation Guidelines**
|
|
||||||
|
|
||||||
- Follow the `instruction` field from `openspec instructions` for each artifact type
|
|
||||||
- The schema defines what each artifact should contain - follow it
|
|
||||||
- Read dependency artifacts for context before creating new ones
|
|
||||||
- Use `template` as the structure for your output file - fill in its sections
|
|
||||||
- **IMPORTANT**: `context` and `rules` are constraints for YOU, not content for the file
|
|
||||||
- Do NOT copy `<context>`, `<rules>`, `<project_context>` blocks into the artifact
|
|
||||||
- These guide what you write, but should never appear in the output
|
|
||||||
|
|
||||||
**Guardrails**
|
|
||||||
- Create ALL artifacts needed for implementation (as defined by schema's `apply.requires`)
|
|
||||||
- Always read dependency artifacts before creating a new one
|
|
||||||
- If context is critically unclear, ask the user - but prefer making reasonable decisions to keep momentum
|
|
||||||
- If a change with that name already exists, ask if user wants to continue it or create a new one
|
|
||||||
- Verify each artifact file exists after writing before proceeding to next
|
|
||||||
@@ -0,0 +1,16 @@
|
|||||||
|
---
|
||||||
|
name: handoff
|
||||||
|
description: Compact the current conversation into a handoff document for another agent to pick up.
|
||||||
|
argument-hint: "What will the next session be used for?"
|
||||||
|
disable-model-invocation: true
|
||||||
|
---
|
||||||
|
|
||||||
|
Write a handoff document summarising the current conversation so a fresh agent can continue the work. Save to the temporary directory of the user's OS - not the current workspace.
|
||||||
|
|
||||||
|
Include a "suggested skills" section in the document, which suggests skills that the agent should invoke.
|
||||||
|
|
||||||
|
Do not duplicate content already captured in other artifacts (PRDs, plans, ADRs, issues, commits, diffs). Reference them by path or URL instead.
|
||||||
|
|
||||||
|
Redact any sensitive information, such as API keys, passwords, or personally identifiable information.
|
||||||
|
|
||||||
|
If the user passed arguments, treat them as a description of what the next session will focus on and tailor the doc accordingly.
|
||||||
@@ -0,0 +1,160 @@
|
|||||||
|
# Handoff: Phase 1 OpenSpec 格式修正
|
||||||
|
|
||||||
|
**交接时间**: 2026-06-23
|
||||||
|
**项目**: SuperBizAgent-java
|
||||||
|
**分支**: emdash/mvp-waq54
|
||||||
|
**任务**: 将 Phase 1 OpenSpec 重构为标准格式
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 背景
|
||||||
|
|
||||||
|
当前正在执行 Phase 1（基础设施搭建）实施，已通过 sm-flow 完整流程生成 OpenSpec，但**格式不符合 OpenSpec 标准规范**。
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 已完成工作
|
||||||
|
|
||||||
|
### 1. Phase 1 代码实施（部分完成）
|
||||||
|
|
||||||
|
**已提交 3 个 commit**：
|
||||||
|
- `5ddb7a6`: Phase 1 基础设施代码
|
||||||
|
- 添加 JPA/Flyway/Redis 依赖到 pom.xml
|
||||||
|
- 创建 3 个 Flyway 迁移脚本（V001/V002/V003）
|
||||||
|
- 创建 3 个枚举类（FaultCategory/DiagnosisStatus/SourceType）
|
||||||
|
- 配置 MySQL + Redis 连接
|
||||||
|
- `3f15778`: Phase 1 文档和 OpenSpec（**格式错误，需要修正**）
|
||||||
|
- `a3d806e`: .gitignore 更新
|
||||||
|
|
||||||
|
**已推送到远程**：`origin/emdash/mvp-waq54`
|
||||||
|
|
||||||
|
**配置信息**（已完成）：
|
||||||
|
- MySQL: 119.29.78.52:33306/superbiz_agent（用户已解决合并冲突后的新分支）
|
||||||
|
- Redis: 119.29.78.52:6379
|
||||||
|
- application.yml 配置完整（保留原有配置）
|
||||||
|
|
||||||
|
**待完成任务**（Phase 1 剩余）：
|
||||||
|
- Task 1.6-1.11: JPA 实体类、Repository、Redis 会话管理
|
||||||
|
- Task 3.1-3.3: 包名重构（org.example → com.superbiz.agent）
|
||||||
|
- Task 4.1-4.7: 文档管理 CRUD + 混合检索
|
||||||
|
|
||||||
|
### 2. OpenSpec 生成（sm-flow 完整流程）
|
||||||
|
|
||||||
|
通过 sm-flow 完整流程（clarify → context → propose → grill → specify → audit → commit）生成了 Phase 1 OpenSpec，但**格式不符合标准**。
|
||||||
|
|
||||||
|
**当前目录结构**（错误）：
|
||||||
|
```
|
||||||
|
openspec/changes/phase-1-infrastructure/
|
||||||
|
├── proposal.md # ❌ 应合并到 change.md
|
||||||
|
├── design.md # ❌ 应合并到 change.md
|
||||||
|
├── specs/
|
||||||
|
│ └── functional-specs.md # ❌ 应为 specs.md
|
||||||
|
├── tasks.md # ❌ 格式错误（详细文档而非任务列表）
|
||||||
|
├── decisions.md # ✅ 格式可能正确
|
||||||
|
└── .commit # ❌ 非标准文件
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 问题诊断
|
||||||
|
|
||||||
|
### 格式问题清单
|
||||||
|
|
||||||
|
1. **文件结构错误**
|
||||||
|
- proposal.md 和 design.md 应合并为 change.md
|
||||||
|
- specs/functional-specs.md 应改为 specs.md
|
||||||
|
- .commit 文件非标准
|
||||||
|
|
||||||
|
2. **tasks.md 格式错误**（用户明确指出）
|
||||||
|
- 当前：详细的 Markdown 文档（标题、粗体、嵌套、描述、验收标准）
|
||||||
|
- 应该：纯任务列表格式（checkbox 列表）
|
||||||
|
- 示例：`- [ ] Task 1.1: 添加依赖到 pom.xml`
|
||||||
|
|
||||||
|
3. **缺少标准格式规范**
|
||||||
|
- 不清楚 change.md 应包含哪些部分
|
||||||
|
- 不清楚 specs.md 的标准结构
|
||||||
|
- 需要参考 OpenSpec 标准示例
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 下一步行动
|
||||||
|
|
||||||
|
### 主要任务：修正 OpenSpec 格式
|
||||||
|
|
||||||
|
**目标**：将 `openspec/changes/phase-1-infrastructure/` 重构为标准 OpenSpec 格式
|
||||||
|
|
||||||
|
**步骤**：
|
||||||
|
1. **了解标准格式**
|
||||||
|
- 阅读 OpenSpec 规范文档或示例
|
||||||
|
- 明确 change.md、specs.md、tasks.md 的标准结构
|
||||||
|
|
||||||
|
2. **重构文件结构**
|
||||||
|
- 合并 proposal.md + design.md → change.md
|
||||||
|
- 重构 specs/functional-specs.md → specs.md
|
||||||
|
- 重写 tasks.md 为简单的 checkbox 列表
|
||||||
|
- 检查 decisions.md 是否符合标准
|
||||||
|
- 删除 .commit 或确认其用途
|
||||||
|
|
||||||
|
3. **验证格式**
|
||||||
|
- 确认符合 OpenSpec 标准
|
||||||
|
- 提交修正后的 OpenSpec
|
||||||
|
|
||||||
|
**约束**：
|
||||||
|
- 保留所有内容价值，只调整格式
|
||||||
|
- 不修改已实施的代码
|
||||||
|
- 不影响 application.yml 中的现有配置
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 建议技能
|
||||||
|
|
||||||
|
1. **openspec-propose** 或 **openspec-apply-change**
|
||||||
|
查看这些技能生成的 OpenSpec 格式，作为标准参考
|
||||||
|
|
||||||
|
2. **Read**
|
||||||
|
读取现有 OpenSpec 文件内容，理解需要重构的部分
|
||||||
|
|
||||||
|
3. **Write** / **Edit**
|
||||||
|
重构 OpenSpec 文件为标准格式
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 关键文件路径
|
||||||
|
|
||||||
|
**OpenSpec 目录**：
|
||||||
|
- `openspec/changes/phase-1-infrastructure/`（需要重构）
|
||||||
|
|
||||||
|
**参考文档**：
|
||||||
|
- `docs/architecture/implementation-detail.md`（实施计划）
|
||||||
|
- `docs/tables/*.md`（数据库表设计）
|
||||||
|
|
||||||
|
**代码文件**（已完成）：
|
||||||
|
- `pom.xml`
|
||||||
|
- `src/main/resources/application.yml`
|
||||||
|
- `src/main/resources/db/migration/V00*.sql`
|
||||||
|
- `src/main/java/com/superbiz/agent/domain/enums/*.java`
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 环境信息
|
||||||
|
|
||||||
|
- **工作目录**: D:\zhu\worktree\SuperBizAgent-java\emdash\mvp-waq54
|
||||||
|
- **Git 分支**: emdash/mvp-waq54
|
||||||
|
- **平台**: Windows (bash shell)
|
||||||
|
- **Maven**: 可用
|
||||||
|
- **数据库**: MySQL 已配置，数据库 `superbiz_agent` 需要用户创建
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 敏感信息（已编辑）
|
||||||
|
|
||||||
|
- MySQL 密码：已配置在 application.yml（`!Fucker123..`）
|
||||||
|
- Redis：无密码
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 备注
|
||||||
|
|
||||||
|
- 用户已解决分支合并冲突，当前在新分支 `emdash/mvp-waq54`
|
||||||
|
- Phase 1 实施暂停在 OpenSpec 格式修正任务
|
||||||
|
- 修正完成后可继续执行 Task 1.6 及后续任务
|
||||||
@@ -0,0 +1,177 @@
|
|||||||
|
# Handoff: Phase 1 OpenSpec 格式修正
|
||||||
|
|
||||||
|
**交接时间**: 2026-06-23
|
||||||
|
**项目**: SuperBizAgent-java
|
||||||
|
**分支**: emdash/mvp-waq54
|
||||||
|
**任务**: 将 Phase 1 OpenSpec 重构为标准格式
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 背景
|
||||||
|
|
||||||
|
当前正在执行 Phase 1（基础设施搭建）实施，已通过 sm-flow 完整流程生成 OpenSpec，但**格式不符合 OpenSpec 标准规范**。
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 已完成工作
|
||||||
|
|
||||||
|
### 1. Phase 1 代码实施（部分完成）
|
||||||
|
|
||||||
|
**已提交 3 个 commit**：
|
||||||
|
- `5ddb7a6`: Phase 1 基础设施代码
|
||||||
|
- 添加 JPA/Flyway/Redis 依赖到 pom.xml
|
||||||
|
- 创建 3 个 Flyway 迁移脚本（V001/V002/V003）
|
||||||
|
- 创建 3 个枚举类（FaultCategory/DiagnosisStatus/SourceType）
|
||||||
|
- 配置 MySQL + Redis 连接
|
||||||
|
- `3f15778`: Phase 1 文档和 OpenSpec（**格式错误，需要修正**）
|
||||||
|
- `a3d806e`: .gitignore 更新
|
||||||
|
|
||||||
|
**已推送到远程**：`origin/emdash/mvp-waq54`
|
||||||
|
|
||||||
|
**配置信息**（已完成）：
|
||||||
|
- **MySQL**: 119.29.78.52:33306/superbiz_agent
|
||||||
|
- 用户: root
|
||||||
|
- 密码: !Fucker123..
|
||||||
|
- driver: com.mysql.cj.jdbc.Driver
|
||||||
|
- URL参数: useUnicode=true&serverTimezone=Asia/Shanghai&allowPublicKeyRetrieval=true
|
||||||
|
- **Redis**: 119.29.78.52:6379
|
||||||
|
- 无密码
|
||||||
|
- database: 0
|
||||||
|
- timeout: 3000ms
|
||||||
|
- 连接池: max-active=8, max-idle=8, min-idle=0
|
||||||
|
- **JPA**:
|
||||||
|
- ddl-auto: validate（Flyway 管理表结构）
|
||||||
|
- show-sql: true
|
||||||
|
- format_sql: true
|
||||||
|
- dialect: org.hibernate.dialect.MySQL8Dialect
|
||||||
|
- **Flyway**:
|
||||||
|
- enabled: true
|
||||||
|
- baseline-on-migrate: true
|
||||||
|
- locations: classpath:db/migration
|
||||||
|
- application.yml 配置完整（保留原有 Milvus、DashScope、MCP、文档分片、RAG、Prometheus、CLS 等配置）
|
||||||
|
|
||||||
|
**待完成任务**（Phase 1 剩余）：
|
||||||
|
- Task 1.6-1.11: JPA 实体类、Repository、Redis 会话管理
|
||||||
|
- Task 3.1-3.3: 包名重构（org.example → com.superbiz.agent）
|
||||||
|
- Task 4.1-4.7: 文档管理 CRUD + 混合检索
|
||||||
|
|
||||||
|
### 2. OpenSpec 生成（sm-flow 完整流程）
|
||||||
|
|
||||||
|
通过 sm-flow 完整流程（clarify → context → propose → grill → specify → audit → commit）生成了 Phase 1 OpenSpec，但**格式不符合标准**。
|
||||||
|
|
||||||
|
**当前目录结构**（错误）：
|
||||||
|
```
|
||||||
|
openspec/changes/phase-1-infrastructure/
|
||||||
|
├── proposal.md # ❌ 应合并到 change.md
|
||||||
|
├── design.md # ❌ 应合并到 change.md
|
||||||
|
├── specs/
|
||||||
|
│ └── functional-specs.md # ❌ 应为 specs.md
|
||||||
|
├── tasks.md # ❌ 格式错误（详细文档而非任务列表）
|
||||||
|
├── decisions.md # ✅ 格式可能正确
|
||||||
|
└── .commit # ❌ 非标准文件
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 问题诊断
|
||||||
|
|
||||||
|
### 格式问题清单
|
||||||
|
|
||||||
|
1. **文件结构错误**
|
||||||
|
- proposal.md 和 design.md 应合并为 change.md
|
||||||
|
- specs/functional-specs.md 应改为 specs.md
|
||||||
|
- .commit 文件非标准
|
||||||
|
|
||||||
|
2. **tasks.md 格式错误**（用户明确指出）
|
||||||
|
- 当前：详细的 Markdown 文档（标题、粗体、嵌套、描述、验收标准）
|
||||||
|
- 应该：纯任务列表格式（checkbox 列表）
|
||||||
|
- 示例：`- [ ] Task 1.1: 添加依赖到 pom.xml`
|
||||||
|
|
||||||
|
3. **缺少标准格式规范**
|
||||||
|
- 不清楚 change.md 应包含哪些部分
|
||||||
|
- 不清楚 specs.md 的标准结构
|
||||||
|
- 需要参考 OpenSpec 标准示例
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 下一步行动
|
||||||
|
|
||||||
|
### 主要任务：修正 OpenSpec 格式
|
||||||
|
|
||||||
|
**目标**：将 `openspec/changes/phase-1-infrastructure/` 重构为标准 OpenSpec 格式
|
||||||
|
|
||||||
|
**步骤**：
|
||||||
|
1. **了解标准格式**
|
||||||
|
- 阅读 OpenSpec 规范文档或示例
|
||||||
|
- 明确 change.md、specs.md、tasks.md 的标准结构
|
||||||
|
|
||||||
|
2. **重构文件结构**
|
||||||
|
- 合并 proposal.md + design.md → change.md
|
||||||
|
- 重构 specs/functional-specs.md → specs.md
|
||||||
|
- 重写 tasks.md 为简单的 checkbox 列表
|
||||||
|
- 检查 decisions.md 是否符合标准
|
||||||
|
- 删除 .commit 或确认其用途
|
||||||
|
|
||||||
|
3. **验证格式**
|
||||||
|
- 确认符合 OpenSpec 标准
|
||||||
|
- 提交修正后的 OpenSpec
|
||||||
|
|
||||||
|
**约束**：
|
||||||
|
- 保留所有内容价值，只调整格式
|
||||||
|
- 不修改已实施的代码
|
||||||
|
- 不影响 application.yml 中的现有配置
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 建议技能
|
||||||
|
|
||||||
|
1. **openspec-propose** 或 **openspec-apply-change**
|
||||||
|
查看这些技能生成的 OpenSpec 格式，作为标准参考
|
||||||
|
|
||||||
|
2. **Read**
|
||||||
|
读取现有 OpenSpec 文件内容，理解需要重构的部分
|
||||||
|
|
||||||
|
3. **Write** / **Edit**
|
||||||
|
重构 OpenSpec 文件为标准格式
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 关键文件路径
|
||||||
|
|
||||||
|
**OpenSpec 目录**：
|
||||||
|
- `openspec/changes/phase-1-infrastructure/`（需要重构）
|
||||||
|
|
||||||
|
**参考文档**：
|
||||||
|
- `docs/architecture/implementation-detail.md`（实施计划）
|
||||||
|
- `docs/tables/*.md`（数据库表设计）
|
||||||
|
|
||||||
|
**代码文件**（已完成）：
|
||||||
|
- `pom.xml`
|
||||||
|
- `src/main/resources/application.yml`
|
||||||
|
- `src/main/resources/db/migration/V00*.sql`
|
||||||
|
- `src/main/java/com/superbiz/agent/domain/enums/*.java`
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 环境信息
|
||||||
|
|
||||||
|
- **工作目录**: D:\zhu\worktree\SuperBizAgent-java\emdash\mvp-waq54
|
||||||
|
- **Git 分支**: emdash/mvp-waq54
|
||||||
|
- **平台**: Windows (bash shell)
|
||||||
|
- **Maven**: 可用
|
||||||
|
- **数据库**: MySQL 已配置，数据库 `superbiz_agent` 需要用户创建
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 敏感信息（已编辑）
|
||||||
|
|
||||||
|
- MySQL 密码：已配置在 application.yml（`!Fucker123..`）
|
||||||
|
- Redis：无密码
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 备注
|
||||||
|
|
||||||
|
- 用户已解决分支合并冲突，当前在新分支 `emdash/mvp-waq54`
|
||||||
|
- Phase 1 实施暂停在 OpenSpec 格式修正任务
|
||||||
|
- 修正完成后可继续执行 Task 1.6 及后续任务
|
||||||
@@ -17,17 +17,17 @@
|
|||||||
- [x] 2.5 创建 CaseLibraryRepository 接口
|
- [x] 2.5 创建 CaseLibraryRepository 接口
|
||||||
- [x] 2.6 创建 ApiDocumentRepository 接口
|
- [x] 2.6 创建 ApiDocumentRepository 接口
|
||||||
- [x] 2.7 Repository 单元测试 (DiagnosisRecordRepositoryTest)
|
- [x] 2.7 Repository 单元测试 (DiagnosisRecordRepositoryTest)
|
||||||
- [ ] 2.8 Repository 单元测试 (CaseLibraryRepositoryTest)
|
- [x] 2.8 Repository 单元测试 (CaseLibraryRepositoryTest)
|
||||||
- [ ] 2.9 Repository 单元测试 (ApiDocumentRepositoryTest)
|
- [x] 2.9 Repository 单元测试 (ApiDocumentRepositoryTest)
|
||||||
|
|
||||||
## 3. 会话管理
|
## 3. 会话管理
|
||||||
|
|
||||||
- [ ] 3.1 创建 SessionManager 接口
|
- [x] 3.1 创建 SessionManager 接口
|
||||||
- [ ] 3.2 创建 SessionContext 数据类
|
- [x] 3.2 创建 SessionContext 数据类
|
||||||
- [ ] 3.3 创建 ToolCall 数据类
|
- [x] 3.3 创建 ToolCall 数据类
|
||||||
- [ ] 3.4 创建 RedisSessionManager 实现
|
- [x] 3.4 创建 RedisSessionManager 实现
|
||||||
- [ ] 3.5 创建 SessionConfiguration 配置类
|
- [x] 3.5 创建 SessionConfiguration 配置类
|
||||||
- [ ] 3.6 Redis 会话管理单元测试 (RedisSessionManagerTest)
|
- [x] 3.6 Redis 会话管理单元测试 (RedisSessionManagerTest)
|
||||||
|
|
||||||
## 4. 代码结构重构
|
## 4. 代码结构重构
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1,55 @@
|
|||||||
|
package org.example.config;
|
||||||
|
|
||||||
|
import com.fasterxml.jackson.annotation.JsonTypeInfo;
|
||||||
|
import com.fasterxml.jackson.databind.ObjectMapper;
|
||||||
|
import com.fasterxml.jackson.databind.jsontype.impl.LaissezFaireSubTypeValidator;
|
||||||
|
import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;
|
||||||
|
import org.springframework.context.annotation.Bean;
|
||||||
|
import org.springframework.context.annotation.Configuration;
|
||||||
|
import org.springframework.data.redis.connection.RedisConnectionFactory;
|
||||||
|
import org.springframework.data.redis.core.RedisTemplate;
|
||||||
|
import org.springframework.data.redis.serializer.GenericJackson2JsonRedisSerializer;
|
||||||
|
import org.springframework.data.redis.serializer.StringRedisSerializer;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Redis 会话配置
|
||||||
|
*/
|
||||||
|
@Configuration
|
||||||
|
public class SessionConfiguration {
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 配置 RedisTemplate
|
||||||
|
* 使用 JSON 序列化存储会话对象
|
||||||
|
*/
|
||||||
|
@Bean
|
||||||
|
public RedisTemplate<String, Object> redisTemplate(RedisConnectionFactory connectionFactory) {
|
||||||
|
RedisTemplate<String, Object> template = new RedisTemplate<>();
|
||||||
|
template.setConnectionFactory(connectionFactory);
|
||||||
|
|
||||||
|
// 配置 ObjectMapper 支持 Java 8 时间类型
|
||||||
|
ObjectMapper objectMapper = new ObjectMapper();
|
||||||
|
objectMapper.registerModule(new JavaTimeModule());
|
||||||
|
|
||||||
|
// 启用类型信息，支持多态反序列化
|
||||||
|
objectMapper.activateDefaultTyping(
|
||||||
|
LaissezFaireSubTypeValidator.instance,
|
||||||
|
ObjectMapper.DefaultTyping.NON_FINAL,
|
||||||
|
JsonTypeInfo.As.PROPERTY
|
||||||
|
);
|
||||||
|
|
||||||
|
// 使用 JSON 序列化器
|
||||||
|
GenericJackson2JsonRedisSerializer jsonSerializer =
|
||||||
|
new GenericJackson2JsonRedisSerializer(objectMapper);
|
||||||
|
|
||||||
|
// Key 使用 String 序列化
|
||||||
|
template.setKeySerializer(new StringRedisSerializer());
|
||||||
|
template.setHashKeySerializer(new StringRedisSerializer());
|
||||||
|
|
||||||
|
// Value 使用 JSON 序列化
|
||||||
|
template.setValueSerializer(jsonSerializer);
|
||||||
|
template.setHashValueSerializer(jsonSerializer);
|
||||||
|
|
||||||
|
template.afterPropertiesSet();
|
||||||
|
return template;
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,88 @@
|
|||||||
|
package org.example.domain.model;
|
||||||
|
|
||||||
|
import lombok.AllArgsConstructor;
|
||||||
|
import lombok.Builder;
|
||||||
|
import lombok.Data;
|
||||||
|
import lombok.NoArgsConstructor;
|
||||||
|
|
||||||
|
import java.io.Serializable;
|
||||||
|
import java.time.LocalDateTime;
|
||||||
|
import java.util.ArrayList;
|
||||||
|
import java.util.List;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 会话上下文数据类
|
||||||
|
* 存储在 Redis 中的会话数据
|
||||||
|
*/
|
||||||
|
@Data
|
||||||
|
@Builder
|
||||||
|
@NoArgsConstructor
|
||||||
|
@AllArgsConstructor
|
||||||
|
public class SessionContext implements Serializable {
|
||||||
|
|
||||||
|
private static final long serialVersionUID = 1L;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 会话ID
|
||||||
|
*/
|
||||||
|
private String sessionId;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 用户ID
|
||||||
|
*/
|
||||||
|
private String userId;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 业务ID（订单号/请求ID等）
|
||||||
|
*/
|
||||||
|
private String businessId;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 链路追踪ID
|
||||||
|
*/
|
||||||
|
private String traceId;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 会话状态（ACTIVE/COMPLETED/EXPIRED）
|
||||||
|
*/
|
||||||
|
private String status;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 工具调用历史
|
||||||
|
*/
|
||||||
|
@Builder.Default
|
||||||
|
private List<ToolCall> toolCalls = new ArrayList<>();
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 会话创建时间
|
||||||
|
*/
|
||||||
|
private LocalDateTime createdAt;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 最后活跃时间
|
||||||
|
*/
|
||||||
|
private LocalDateTime lastActiveAt;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 会话过期时间（秒）
|
||||||
|
*/
|
||||||
|
private Long ttl;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 添加工具调用记录
|
||||||
|
*/
|
||||||
|
public void addToolCall(ToolCall toolCall) {
|
||||||
|
if (this.toolCalls == null) {
|
||||||
|
this.toolCalls = new ArrayList<>();
|
||||||
|
}
|
||||||
|
this.toolCalls.add(toolCall);
|
||||||
|
this.lastActiveAt = LocalDateTime.now();
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 更新最后活跃时间
|
||||||
|
*/
|
||||||
|
public void updateLastActiveTime() {
|
||||||
|
this.lastActiveAt = LocalDateTime.now();
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,57 @@
|
|||||||
|
package org.example.domain.model;
|
||||||
|
|
||||||
|
import lombok.AllArgsConstructor;
|
||||||
|
import lombok.Builder;
|
||||||
|
import lombok.Data;
|
||||||
|
import lombok.NoArgsConstructor;
|
||||||
|
|
||||||
|
import java.io.Serializable;
|
||||||
|
import java.time.LocalDateTime;
|
||||||
|
import java.util.Map;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 工具调用记录数据类
|
||||||
|
*/
|
||||||
|
@Data
|
||||||
|
@Builder
|
||||||
|
@NoArgsConstructor
|
||||||
|
@AllArgsConstructor
|
||||||
|
public class ToolCall implements Serializable {
|
||||||
|
|
||||||
|
private static final long serialVersionUID = 1L;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 工具名称
|
||||||
|
*/
|
||||||
|
private String toolName;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 工具参数
|
||||||
|
*/
|
||||||
|
private Map<String, Object> arguments;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 工具返回结果
|
||||||
|
*/
|
||||||
|
private String result;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 执行状态（SUCCESS/FAILED/TIMEOUT）
|
||||||
|
*/
|
||||||
|
private String status;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 错误信息（如果失败）
|
||||||
|
*/
|
||||||
|
private String errorMessage;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 执行耗时（毫秒）
|
||||||
|
*/
|
||||||
|
private Long duration;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 调用时间
|
||||||
|
*/
|
||||||
|
private LocalDateTime calledAt;
|
||||||
|
}
|
||||||
@@ -0,0 +1,77 @@
|
|||||||
|
package org.example.service.session;
|
||||||
|
|
||||||
|
import org.example.domain.model.SessionContext;
|
||||||
|
import org.example.domain.model.ToolCall;
|
||||||
|
|
||||||
|
import java.util.Optional;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 会话管理器接口
|
||||||
|
* 负责会话的创建、读取、更新和删除
|
||||||
|
*/
|
||||||
|
public interface SessionManager {
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 创建新会话
|
||||||
|
*
|
||||||
|
* @param sessionContext 会话上下文
|
||||||
|
* @param ttlSeconds 会话过期时间（秒）
|
||||||
|
* @return 会话ID
|
||||||
|
*/
|
||||||
|
String createSession(SessionContext sessionContext, long ttlSeconds);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 获取会话
|
||||||
|
*
|
||||||
|
* @param sessionId 会话ID
|
||||||
|
* @return 会话上下文（如果存在）
|
||||||
|
*/
|
||||||
|
Optional<SessionContext> getSession(String sessionId);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 更新会话
|
||||||
|
*
|
||||||
|
* @param sessionContext 会话上下文
|
||||||
|
*/
|
||||||
|
void updateSession(SessionContext sessionContext);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 删除会话
|
||||||
|
*
|
||||||
|
* @param sessionId 会话ID
|
||||||
|
*/
|
||||||
|
void deleteSession(String sessionId);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 检查会话是否存在
|
||||||
|
*
|
||||||
|
* @param sessionId 会话ID
|
||||||
|
* @return true 如果会话存在
|
||||||
|
*/
|
||||||
|
boolean exists(String sessionId);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 刷新会话过期时间
|
||||||
|
*
|
||||||
|
* @param sessionId 会话ID
|
||||||
|
* @param ttlSeconds 新的过期时间（秒）
|
||||||
|
* @return true 如果刷新成功
|
||||||
|
*/
|
||||||
|
boolean refreshSession(String sessionId, long ttlSeconds);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 添加工具调用记录到会话
|
||||||
|
*
|
||||||
|
* @param sessionId 会话ID
|
||||||
|
* @param toolCall 工具调用记录
|
||||||
|
*/
|
||||||
|
void addToolCall(String sessionId, ToolCall toolCall);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 更新会话状态
|
||||||
|
*
|
||||||
|
* @param sessionId 会话ID
|
||||||
|
* @param status 新状态
|
||||||
|
*/
|
||||||
|
void updateStatus(String sessionId, String status);
|
||||||
|
}
|
||||||
@@ -0,0 +1,147 @@
|
|||||||
|
package org.example.service.session.impl;
|
||||||
|
|
||||||
|
import lombok.RequiredArgsConstructor;
|
||||||
|
import lombok.extern.slf4j.Slf4j;
|
||||||
|
import org.example.domain.model.SessionContext;
|
||||||
|
import org.example.domain.model.ToolCall;
|
||||||
|
import org.example.service.session.SessionManager;
|
||||||
|
import org.springframework.data.redis.core.RedisTemplate;
|
||||||
|
import org.springframework.stereotype.Service;
|
||||||
|
|
||||||
|
import java.time.LocalDateTime;
|
||||||
|
import java.util.Optional;
|
||||||
|
import java.util.concurrent.TimeUnit;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Redis 会话管理器实现
|
||||||
|
*/
|
||||||
|
@Slf4j
|
||||||
|
@Service
|
||||||
|
@RequiredArgsConstructor
|
||||||
|
public class RedisSessionManager implements SessionManager {
|
||||||
|
|
||||||
|
private static final String SESSION_KEY_PREFIX = "session:";
|
||||||
|
|
||||||
|
private final RedisTemplate<String, Object> redisTemplate;
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String createSession(SessionContext sessionContext, long ttlSeconds) {
|
||||||
|
String sessionId = sessionContext.getSessionId();
|
||||||
|
if (sessionId == null || sessionId.isEmpty()) {
|
||||||
|
throw new IllegalArgumentException("Session ID cannot be null or empty");
|
||||||
|
}
|
||||||
|
|
||||||
|
sessionContext.setCreatedAt(LocalDateTime.now());
|
||||||
|
sessionContext.setLastActiveAt(LocalDateTime.now());
|
||||||
|
sessionContext.setTtl(ttlSeconds);
|
||||||
|
sessionContext.setStatus("ACTIVE");
|
||||||
|
|
||||||
|
String key = buildKey(sessionId);
|
||||||
|
redisTemplate.opsForValue().set(key, sessionContext, ttlSeconds, TimeUnit.SECONDS);
|
||||||
|
|
||||||
|
log.info("创建会话成功: sessionId={}, ttl={}秒", sessionId, ttlSeconds);
|
||||||
|
return sessionId;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Optional<SessionContext> getSession(String sessionId) {
|
||||||
|
String key = buildKey(sessionId);
|
||||||
|
Object value = redisTemplate.opsForValue().get(key);
|
||||||
|
|
||||||
|
if (value instanceof SessionContext) {
|
||||||
|
SessionContext context = (SessionContext) value;
|
||||||
|
log.debug("获取会话成功: sessionId={}", sessionId);
|
||||||
|
return Optional.of(context);
|
||||||
|
}
|
||||||
|
|
||||||
|
log.debug("会话不存在: sessionId={}", sessionId);
|
||||||
|
return Optional.empty();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void updateSession(SessionContext sessionContext) {
|
||||||
|
String sessionId = sessionContext.getSessionId();
|
||||||
|
String key = buildKey(sessionId);
|
||||||
|
|
||||||
|
// 获取剩余 TTL
|
||||||
|
Long ttl = redisTemplate.getExpire(key, TimeUnit.SECONDS);
|
||||||
|
if (ttl == null || ttl <= 0) {
|
||||||
|
ttl = sessionContext.getTtl() != null ? sessionContext.getTtl() : 3600L;
|
||||||
|
}
|
||||||
|
|
||||||
|
sessionContext.setLastActiveAt(LocalDateTime.now());
|
||||||
|
redisTemplate.opsForValue().set(key, sessionContext, ttl, TimeUnit.SECONDS);
|
||||||
|
|
||||||
|
log.debug("更新会话成功: sessionId={}", sessionId);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void deleteSession(String sessionId) {
|
||||||
|
String key = buildKey(sessionId);
|
||||||
|
Boolean deleted = redisTemplate.delete(key);
|
||||||
|
|
||||||
|
if (Boolean.TRUE.equals(deleted)) {
|
||||||
|
log.info("删除会话成功: sessionId={}", sessionId);
|
||||||
|
} else {
|
||||||
|
log.warn("删除会话失败，会话可能不存在: sessionId={}", sessionId);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean exists(String sessionId) {
|
||||||
|
String key = buildKey(sessionId);
|
||||||
|
Boolean exists = redisTemplate.hasKey(key);
|
||||||
|
return Boolean.TRUE.equals(exists);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean refreshSession(String sessionId, long ttlSeconds) {
|
||||||
|
String key = buildKey(sessionId);
|
||||||
|
Boolean refreshed = redisTemplate.expire(key, ttlSeconds, TimeUnit.SECONDS);
|
||||||
|
|
||||||
|
if (Boolean.TRUE.equals(refreshed)) {
|
||||||
|
log.debug("刷新会话过期时间成功: sessionId={}, newTtl={}秒", sessionId, ttlSeconds);
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
log.warn("刷新会话过期时间失败，会话可能不存在: sessionId={}", sessionId);
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void addToolCall(String sessionId, ToolCall toolCall) {
|
||||||
|
Optional<SessionContext> sessionOpt = getSession(sessionId);
|
||||||
|
|
||||||
|
if (sessionOpt.isPresent()) {
|
||||||
|
SessionContext context = sessionOpt.get();
|
||||||
|
context.addToolCall(toolCall);
|
||||||
|
updateSession(context);
|
||||||
|
|
||||||
|
log.debug("添加工具调用记录成功: sessionId={}, toolName={}", sessionId, toolCall.getToolName());
|
||||||
|
} else {
|
||||||
|
log.warn("会话不存在，无法添加工具调用记录: sessionId={}", sessionId);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void updateStatus(String sessionId, String status) {
|
||||||
|
Optional<SessionContext> sessionOpt = getSession(sessionId);
|
||||||
|
|
||||||
|
if (sessionOpt.isPresent()) {
|
||||||
|
SessionContext context = sessionOpt.get();
|
||||||
|
context.setStatus(status);
|
||||||
|
updateSession(context);
|
||||||
|
|
||||||
|
log.debug("更新会话状态成功: sessionId={}, status={}", sessionId, status);
|
||||||
|
} else {
|
||||||
|
log.warn("会话不存在，无法更新状态: sessionId={}", sessionId);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 构建 Redis key
|
||||||
|
*/
|
||||||
|
private String buildKey(String sessionId) {
|
||||||
|
return SESSION_KEY_PREFIX + sessionId;
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,64 @@
|
|||||||
|
package org.example.config;
|
||||||
|
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
import org.springframework.beans.factory.annotation.Autowired;
|
||||||
|
import org.springframework.boot.test.context.SpringBootTest;
|
||||||
|
import org.springframework.data.redis.core.RedisTemplate;
|
||||||
|
|
||||||
|
import javax.sql.DataSource;
|
||||||
|
import java.sql.Connection;
|
||||||
|
|
||||||
|
import static org.junit.jupiter.api.Assertions.*;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 测试 MySQL 和 Redis 连接配置
|
||||||
|
*/
|
||||||
|
@SpringBootTest
|
||||||
|
class ConnectionConfigTest {
|
||||||
|
|
||||||
|
@Autowired(required = false)
|
||||||
|
private DataSource dataSource;
|
||||||
|
|
||||||
|
@Autowired(required = false)
|
||||||
|
private RedisTemplate<String, Object> redisTemplate;
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testMySQLConnection() {
|
||||||
|
assertNotNull(dataSource, "DataSource should be configured");
|
||||||
|
|
||||||
|
try (Connection connection = dataSource.getConnection()) {
|
||||||
|
assertNotNull(connection, "Connection should not be null");
|
||||||
|
assertFalse(connection.isClosed(), "Connection should be open");
|
||||||
|
|
||||||
|
String catalog = connection.getCatalog();
|
||||||
|
System.out.println("✓ MySQL 连接成功！数据库: " + catalog);
|
||||||
|
assertEquals("superbiz_agent", catalog, "Database name should be superbiz_agent");
|
||||||
|
|
||||||
|
} catch (Exception e) {
|
||||||
|
fail("MySQL 连接失败: " + e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testRedisConnection() {
|
||||||
|
assertNotNull(redisTemplate, "RedisTemplate should be configured");
|
||||||
|
|
||||||
|
try {
|
||||||
|
// 测试 PING
|
||||||
|
String testKey = "test:connection:" + System.currentTimeMillis();
|
||||||
|
String testValue = "test-value";
|
||||||
|
|
||||||
|
redisTemplate.opsForValue().set(testKey, testValue);
|
||||||
|
Object result = redisTemplate.opsForValue().get(testKey);
|
||||||
|
|
||||||
|
assertEquals(testValue, result, "Redis read/write should work");
|
||||||
|
System.out.println("✓ Redis 连接成功！读写正常");
|
||||||
|
|
||||||
|
// 清理测试数据
|
||||||
|
redisTemplate.delete(testKey);
|
||||||
|
|
||||||
|
} catch (Exception e) {
|
||||||
|
fail("Redis 连接失败: " + e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,71 @@
|
|||||||
|
package org.example.config;
|
||||||
|
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
import org.springframework.beans.factory.annotation.Autowired;
|
||||||
|
import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase;
|
||||||
|
import org.springframework.boot.test.autoconfigure.orm.jpa.DataJpaTest;
|
||||||
|
import org.springframework.test.context.TestPropertySource;
|
||||||
|
|
||||||
|
import javax.sql.DataSource;
|
||||||
|
import java.sql.Connection;
|
||||||
|
|
||||||
|
import static org.junit.jupiter.api.Assertions.*;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 单独测试 MySQL 连接（不启动完整 Spring Context）
|
||||||
|
*/
|
||||||
|
@DataJpaTest
|
||||||
|
@AutoConfigureTestDatabase(replace = AutoConfigureTestDatabase.Replace.NONE)
|
||||||
|
@TestPropertySource(properties = {
|
||||||
|
"spring.flyway.enabled=true",
|
||||||
|
"spring.jpa.hibernate.ddl-auto=validate"
|
||||||
|
})
|
||||||
|
class MySQLConnectionTest {
|
||||||
|
|
||||||
|
@Autowired
|
||||||
|
private DataSource dataSource;
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testMySQLConnection() {
|
||||||
|
assertNotNull(dataSource, "DataSource should be configured");
|
||||||
|
|
||||||
|
try (Connection connection = dataSource.getConnection()) {
|
||||||
|
assertNotNull(connection, "Connection should not be null");
|
||||||
|
assertFalse(connection.isClosed(), "Connection should be open");
|
||||||
|
|
||||||
|
String catalog = connection.getCatalog();
|
||||||
|
System.out.println("✓ MySQL 连接成功！");
|
||||||
|
System.out.println(" 数据库: " + catalog);
|
||||||
|
System.out.println(" URL: " + connection.getMetaData().getURL());
|
||||||
|
|
||||||
|
assertEquals("superbiz_agent", catalog, "Database name should be superbiz_agent");
|
||||||
|
|
||||||
|
} catch (Exception e) {
|
||||||
|
fail("MySQL 连接失败: " + e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFlywayMigration() {
|
||||||
|
// 如果能到这里，说明 Flyway 迁移成功
|
||||||
|
System.out.println("✓ Flyway 迁移已执行");
|
||||||
|
|
||||||
|
try (Connection connection = dataSource.getConnection()) {
|
||||||
|
var metadata = connection.getMetaData();
|
||||||
|
var tables = metadata.getTables(null, null, "%", new String[]{"TABLE"});
|
||||||
|
|
||||||
|
int tableCount = 0;
|
||||||
|
System.out.println(" 已创建的表:");
|
||||||
|
while (tables.next()) {
|
||||||
|
String tableName = tables.getString("TABLE_NAME");
|
||||||
|
System.out.println(" - " + tableName);
|
||||||
|
tableCount++;
|
||||||
|
}
|
||||||
|
|
||||||
|
assertTrue(tableCount >= 3, "至少应该有 3 张表 (diagnosis_record, case_library, api_document)");
|
||||||
|
|
||||||
|
} catch (Exception e) {
|
||||||
|
fail("验证表结构失败: " + e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,55 @@
|
|||||||
|
package org.example.config;
|
||||||
|
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
import org.springframework.beans.factory.annotation.Autowired;
|
||||||
|
import org.springframework.boot.test.context.SpringBootTest;
|
||||||
|
import org.springframework.data.redis.core.RedisTemplate;
|
||||||
|
import org.springframework.test.context.TestPropertySource;
|
||||||
|
|
||||||
|
import static org.junit.jupiter.api.Assertions.*;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 单独测试 Redis 连接
|
||||||
|
* 禁用 Milvus 以避免启动失败
|
||||||
|
*/
|
||||||
|
@SpringBootTest
|
||||||
|
@TestPropertySource(properties = {
|
||||||
|
"spring.autoconfigure.exclude=org.example.config.MilvusConfig"
|
||||||
|
})
|
||||||
|
class RedisConnectionTest {
|
||||||
|
|
||||||
|
@Autowired(required = false)
|
||||||
|
private RedisTemplate<String, Object> redisTemplate;
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testRedisConnection() {
|
||||||
|
assertNotNull(redisTemplate, "RedisTemplate should be configured");
|
||||||
|
|
||||||
|
try {
|
||||||
|
String testKey = "test:connection:" + System.currentTimeMillis();
|
||||||
|
String testValue = "test-value";
|
||||||
|
|
||||||
|
// 测试写入
|
||||||
|
redisTemplate.opsForValue().set(testKey, testValue);
|
||||||
|
System.out.println("✓ Redis 写入成功");
|
||||||
|
|
||||||
|
// 测试读取
|
||||||
|
Object result = redisTemplate.opsForValue().get(testKey);
|
||||||
|
assertEquals(testValue, result, "Redis read should return the written value");
|
||||||
|
System.out.println("✓ Redis 读取成功");
|
||||||
|
|
||||||
|
// 测试删除
|
||||||
|
redisTemplate.delete(testKey);
|
||||||
|
Object deleted = redisTemplate.opsForValue().get(testKey);
|
||||||
|
assertNull(deleted, "Deleted key should return null");
|
||||||
|
System.out.println("✓ Redis 删除成功");
|
||||||
|
|
||||||
|
System.out.println("\n✓ Redis 连接测试通过！");
|
||||||
|
System.out.println(" 服务器: 119.29.78.52:6379");
|
||||||
|
System.out.println(" 读写删操作正常");
|
||||||
|
|
||||||
|
} catch (Exception e) {
|
||||||
|
fail("Redis 连接失败: " + e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,50 @@
|
|||||||
|
package org.example.config;
|
||||||
|
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
import org.springframework.beans.factory.annotation.Autowired;
|
||||||
|
import org.springframework.boot.test.context.SpringBootTest;
|
||||||
|
import org.springframework.data.redis.core.StringRedisTemplate;
|
||||||
|
|
||||||
|
import static org.junit.jupiter.api.Assertions.*;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Redis 连接测试
|
||||||
|
* 注意：如果 Milvus 未启动，此测试会因 Spring Context 加载失败而失败
|
||||||
|
* 使用 @DataRedisTest 可以避免加载完整 Context，但需要额外配置
|
||||||
|
*/
|
||||||
|
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.NONE)
|
||||||
|
class SimpleRedisTest {
|
||||||
|
|
||||||
|
@Autowired(required = false)
|
||||||
|
private StringRedisTemplate stringRedisTemplate;
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testRedisBasicOperations() {
|
||||||
|
// 如果能到这里，说明 Spring Context 加载成功，Redis 配置正确
|
||||||
|
assertNotNull(stringRedisTemplate, "StringRedisTemplate should be configured");
|
||||||
|
|
||||||
|
String testKey = "test:simple:" + System.currentTimeMillis();
|
||||||
|
String testValue = "hello-redis";
|
||||||
|
|
||||||
|
try {
|
||||||
|
// 写入
|
||||||
|
stringRedisTemplate.opsForValue().set(testKey, testValue);
|
||||||
|
System.out.println("✓ Redis 写入成功: " + testKey + " = " + testValue);
|
||||||
|
|
||||||
|
// 读取
|
||||||
|
String result = stringRedisTemplate.opsForValue().get(testKey);
|
||||||
|
assertEquals(testValue, result);
|
||||||
|
System.out.println("✓ Redis 读取成功: " + result);
|
||||||
|
|
||||||
|
// 删除
|
||||||
|
Boolean deleted = stringRedisTemplate.delete(testKey);
|
||||||
|
assertTrue(deleted != null && deleted);
|
||||||
|
System.out.println("✓ Redis 删除成功");
|
||||||
|
|
||||||
|
System.out.println("\n✅ Redis 连接正常，读写删操作成功");
|
||||||
|
|
||||||
|
} catch (Exception e) {
|
||||||
|
fail("Redis 操作失败: " + e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,179 @@
|
|||||||
|
package org.example.repository;
|
||||||
|
|
||||||
|
import com.superbiz.agent.domain.enums.FaultCategory;
|
||||||
|
import org.example.domain.entity.ApiDocument;
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
import org.springframework.beans.factory.annotation.Autowired;
|
||||||
|
import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase;
|
||||||
|
import org.springframework.boot.test.autoconfigure.orm.jpa.DataJpaTest;
|
||||||
|
import org.springframework.data.domain.Page;
|
||||||
|
import org.springframework.data.domain.PageRequest;
|
||||||
|
import org.springframework.test.context.TestPropertySource;
|
||||||
|
|
||||||
|
import java.time.LocalDateTime;
|
||||||
|
import java.util.List;
|
||||||
|
import java.util.Optional;
|
||||||
|
import java.util.UUID;
|
||||||
|
|
||||||
|
import static org.junit.jupiter.api.Assertions.*;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* ApiDocumentRepository 单元测试
|
||||||
|
*/
|
||||||
|
@DataJpaTest
|
||||||
|
@AutoConfigureTestDatabase(replace = AutoConfigureTestDatabase.Replace.NONE)
|
||||||
|
@TestPropertySource(properties = {
|
||||||
|
"spring.flyway.enabled=true",
|
||||||
|
"spring.jpa.hibernate.ddl-auto=validate",
|
||||||
|
"spring.jpa.show-sql=false"
|
||||||
|
})
|
||||||
|
class ApiDocumentRepositoryTest {
|
||||||
|
|
||||||
|
@Autowired
|
||||||
|
private ApiDocumentRepository repository;
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testSaveAndFindById() {
|
||||||
|
ApiDocument doc = ApiDocument.builder()
|
||||||
|
.docId(UUID.randomUUID().toString())
|
||||||
|
.fileName("api-spec.md")
|
||||||
|
.filePath("/uploads/api-spec.md")
|
||||||
|
.fileHash("abc123hash")
|
||||||
|
.fileSize(1024L)
|
||||||
|
.faultCategory(FaultCategory.EXTERNAL_API)
|
||||||
|
.faultSource("广东")
|
||||||
|
.apiName("查询接口")
|
||||||
|
.status("PENDING")
|
||||||
|
.chunkCount(0)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
ApiDocument saved = repository.save(doc);
|
||||||
|
assertNotNull(saved.getId());
|
||||||
|
assertNotNull(saved.getCreatedAt());
|
||||||
|
System.out.println("✓ 保存文档成功，ID: " + saved.getId());
|
||||||
|
|
||||||
|
Optional<ApiDocument> found = repository.findById(saved.getId());
|
||||||
|
assertTrue(found.isPresent());
|
||||||
|
assertEquals("api-spec.md", found.get().getFileName());
|
||||||
|
System.out.println("✓ 查询文档成功");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindByDocId() {
|
||||||
|
String docId = UUID.randomUUID().toString();
|
||||||
|
ApiDocument doc = ApiDocument.builder()
|
||||||
|
.docId(docId)
|
||||||
|
.fileName("test-doc.txt")
|
||||||
|
.status("INDEXED")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
repository.save(doc);
|
||||||
|
|
||||||
|
Optional<ApiDocument> found = repository.findByDocId(docId);
|
||||||
|
assertTrue(found.isPresent());
|
||||||
|
assertEquals(docId, found.get().getDocId());
|
||||||
|
System.out.println("✓ 根据 docId 查询成功");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindByFileHash() {
|
||||||
|
String fileHash = "unique-hash-" + System.currentTimeMillis();
|
||||||
|
ApiDocument doc = ApiDocument.builder()
|
||||||
|
.docId(UUID.randomUUID().toString())
|
||||||
|
.fileName("duplicate-check.md")
|
||||||
|
.fileHash(fileHash)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
repository.save(doc);
|
||||||
|
|
||||||
|
Optional<ApiDocument> found = repository.findByFileHash(fileHash);
|
||||||
|
assertTrue(found.isPresent());
|
||||||
|
assertEquals(fileHash, found.get().getFileHash());
|
||||||
|
System.out.println("✓ 根据 fileHash 查询成功（去重检测）");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindByStatus() {
|
||||||
|
ApiDocument doc1 = ApiDocument.builder()
|
||||||
|
.docId(UUID.randomUUID().toString())
|
||||||
|
.fileName("pending-1.md")
|
||||||
|
.status("PENDING")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
ApiDocument doc2 = ApiDocument.builder()
|
||||||
|
.docId(UUID.randomUUID().toString())
|
||||||
|
.fileName("pending-2.md")
|
||||||
|
.status("PENDING")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
repository.save(doc1);
|
||||||
|
repository.save(doc2);
|
||||||
|
|
||||||
|
List<ApiDocument> results = repository.findByStatus("PENDING");
|
||||||
|
assertFalse(results.isEmpty());
|
||||||
|
assertTrue(results.size() >= 2);
|
||||||
|
System.out.println("✓ 根据状态查询成功，找到 " + results.size() + " 条 PENDING 文档");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindByStatusWithPagination() {
|
||||||
|
// 创建多个文档
|
||||||
|
for (int i = 0; i < 5; i++) {
|
||||||
|
ApiDocument doc = ApiDocument.builder()
|
||||||
|
.docId(UUID.randomUUID().toString())
|
||||||
|
.fileName("doc-" + i + ".txt")
|
||||||
|
.status("INDEXED")
|
||||||
|
.chunkCount(10)
|
||||||
|
.build();
|
||||||
|
repository.save(doc);
|
||||||
|
}
|
||||||
|
|
||||||
|
Page<ApiDocument> page = repository.findByStatus("INDEXED", PageRequest.of(0, 3));
|
||||||
|
assertFalse(page.isEmpty());
|
||||||
|
assertTrue(page.getContent().size() <= 3);
|
||||||
|
System.out.println("✓ 分页查询成功，本页 " + page.getContent().size() + " 条，总计 " + page.getTotalElements() + " 条");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testUpdateDocumentStatus() {
|
||||||
|
ApiDocument doc = ApiDocument.builder()
|
||||||
|
.docId(UUID.randomUUID().toString())
|
||||||
|
.fileName("indexing-doc.md")
|
||||||
|
.status("PENDING")
|
||||||
|
.chunkCount(0)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
ApiDocument saved = repository.save(doc);
|
||||||
|
Long id = saved.getId();
|
||||||
|
|
||||||
|
// 更新状态
|
||||||
|
saved.setStatus("INDEXED");
|
||||||
|
saved.setChunkCount(15);
|
||||||
|
saved.setIndexedAt(LocalDateTime.now());
|
||||||
|
|
||||||
|
repository.save(saved);
|
||||||
|
|
||||||
|
Optional<ApiDocument> updated = repository.findById(id);
|
||||||
|
assertTrue(updated.isPresent());
|
||||||
|
assertEquals("INDEXED", updated.get().getStatus());
|
||||||
|
assertEquals(15, updated.get().getChunkCount());
|
||||||
|
assertNotNull(updated.get().getIndexedAt());
|
||||||
|
System.out.println("✓ 更新文档状态成功");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindByFaultSource() {
|
||||||
|
ApiDocument doc = ApiDocument.builder()
|
||||||
|
.docId(UUID.randomUUID().toString())
|
||||||
|
.fileName("guangdong-api.md")
|
||||||
|
.faultSource("广东")
|
||||||
|
.faultCategory(FaultCategory.EXTERNAL_API)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
repository.save(doc);
|
||||||
|
|
||||||
|
List<ApiDocument> results = repository.findByFaultSource("广东");
|
||||||
|
assertFalse(results.isEmpty());
|
||||||
|
System.out.println("✓ 根据故障源查询成功，找到 " + results.size() + " 条文档");
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,173 @@
|
|||||||
|
package org.example.repository;
|
||||||
|
|
||||||
|
import com.superbiz.agent.domain.enums.FaultCategory;
|
||||||
|
import com.superbiz.agent.domain.enums.SourceType;
|
||||||
|
import org.example.domain.entity.CaseLibrary;
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
import org.springframework.beans.factory.annotation.Autowired;
|
||||||
|
import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase;
|
||||||
|
import org.springframework.boot.test.autoconfigure.orm.jpa.DataJpaTest;
|
||||||
|
import org.springframework.data.domain.Page;
|
||||||
|
import org.springframework.data.domain.PageRequest;
|
||||||
|
import org.springframework.test.context.TestPropertySource;
|
||||||
|
|
||||||
|
import java.util.List;
|
||||||
|
import java.util.Optional;
|
||||||
|
import java.util.UUID;
|
||||||
|
|
||||||
|
import static org.junit.jupiter.api.Assertions.*;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* CaseLibraryRepository 单元测试
|
||||||
|
*/
|
||||||
|
@DataJpaTest
|
||||||
|
@AutoConfigureTestDatabase(replace = AutoConfigureTestDatabase.Replace.NONE)
|
||||||
|
@TestPropertySource(properties = {
|
||||||
|
"spring.flyway.enabled=true",
|
||||||
|
"spring.jpa.hibernate.ddl-auto=validate",
|
||||||
|
"spring.jpa.show-sql=false"
|
||||||
|
})
|
||||||
|
class CaseLibraryRepositoryTest {
|
||||||
|
|
||||||
|
@Autowired
|
||||||
|
private CaseLibraryRepository repository;
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testSaveAndFindById() {
|
||||||
|
CaseLibrary caseLib = CaseLibrary.builder()
|
||||||
|
.caseId(UUID.randomUUID().toString())
|
||||||
|
.title("接口超时案例")
|
||||||
|
.rootCause("网络延迟导致接口超时")
|
||||||
|
.solution("增加超时时间和重试机制")
|
||||||
|
.faultCategory(FaultCategory.EXTERNAL_API)
|
||||||
|
.errorCode("40003")
|
||||||
|
.sourceType(SourceType.AUTO)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
CaseLibrary saved = repository.save(caseLib);
|
||||||
|
assertNotNull(saved.getId());
|
||||||
|
assertNotNull(saved.getCreatedAt());
|
||||||
|
System.out.println("✓ 保存案例成功，ID: " + saved.getId());
|
||||||
|
|
||||||
|
Optional<CaseLibrary> found = repository.findById(saved.getId());
|
||||||
|
assertTrue(found.isPresent());
|
||||||
|
assertEquals("接口超时案例", found.get().getTitle());
|
||||||
|
System.out.println("✓ 查询案例成功");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindByCaseId() {
|
||||||
|
String caseId = UUID.randomUUID().toString();
|
||||||
|
CaseLibrary caseLib = CaseLibrary.builder()
|
||||||
|
.caseId(caseId)
|
||||||
|
.title("数据库死锁案例")
|
||||||
|
.rootCause("并发更新导致死锁")
|
||||||
|
.solution("优化事务粒度")
|
||||||
|
.faultCategory(FaultCategory.DATABASE)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
repository.save(caseLib);
|
||||||
|
|
||||||
|
Optional<CaseLibrary> found = repository.findByCaseId(caseId);
|
||||||
|
assertTrue(found.isPresent());
|
||||||
|
assertEquals(caseId, found.get().getCaseId());
|
||||||
|
System.out.println("✓ 根据 caseId 查询成功");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindByFaultCategoryAndErrorCode() {
|
||||||
|
CaseLibrary case1 = CaseLibrary.builder()
|
||||||
|
.caseId(UUID.randomUUID().toString())
|
||||||
|
.title("案例1")
|
||||||
|
.rootCause("原因1")
|
||||||
|
.solution("方案1")
|
||||||
|
.faultCategory(FaultCategory.EXTERNAL_API)
|
||||||
|
.errorCode("40003")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
CaseLibrary case2 = CaseLibrary.builder()
|
||||||
|
.caseId(UUID.randomUUID().toString())
|
||||||
|
.title("案例2")
|
||||||
|
.rootCause("原因2")
|
||||||
|
.solution("方案2")
|
||||||
|
.faultCategory(FaultCategory.EXTERNAL_API)
|
||||||
|
.errorCode("40003")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
repository.save(case1);
|
||||||
|
repository.save(case2);
|
||||||
|
|
||||||
|
List<CaseLibrary> results = repository.findByFaultCategoryAndErrorCode(
|
||||||
|
FaultCategory.EXTERNAL_API, "40003");
|
||||||
|
|
||||||
|
assertFalse(results.isEmpty());
|
||||||
|
assertTrue(results.size() >= 2);
|
||||||
|
System.out.println("✓ 根据故障类别和错误码查询成功，找到 " + results.size() + " 条案例");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindBySourceType() {
|
||||||
|
CaseLibrary autoCase = CaseLibrary.builder()
|
||||||
|
.caseId(UUID.randomUUID().toString())
|
||||||
|
.title("自动生成案例")
|
||||||
|
.rootCause("自动分析")
|
||||||
|
.solution("自动方案")
|
||||||
|
.sourceType(SourceType.AUTO)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
repository.save(autoCase);
|
||||||
|
|
||||||
|
Page<CaseLibrary> results = repository.findBySourceType(
|
||||||
|
SourceType.AUTO, PageRequest.of(0, 10));
|
||||||
|
|
||||||
|
assertFalse(results.isEmpty());
|
||||||
|
System.out.println("✓ 根据来源类型查询成功，找到 " + results.getTotalElements() + " 条案例");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testUpdateReferenceCount() {
|
||||||
|
CaseLibrary caseLib = CaseLibrary.builder()
|
||||||
|
.caseId(UUID.randomUUID().toString())
|
||||||
|
.title("热门案例")
|
||||||
|
.rootCause("常见问题")
|
||||||
|
.solution("标准方案")
|
||||||
|
.referenceCount(0)
|
||||||
|
.build();
|
||||||
|
|
||||||
|
CaseLibrary saved = repository.save(caseLib);
|
||||||
|
Long id = saved.getId();
|
||||||
|
|
||||||
|
// 模拟被引用
|
||||||
|
saved.setReferenceCount(saved.getReferenceCount() + 1);
|
||||||
|
repository.save(saved);
|
||||||
|
|
||||||
|
Optional<CaseLibrary> updated = repository.findById(id);
|
||||||
|
assertTrue(updated.isPresent());
|
||||||
|
assertEquals(1, updated.get().getReferenceCount());
|
||||||
|
System.out.println("✓ 更新引用次数成功");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testFindTopByReferenceCount() {
|
||||||
|
// 创建一些案例
|
||||||
|
for (int i = 0; i < 3; i++) {
|
||||||
|
CaseLibrary caseLib = CaseLibrary.builder()
|
||||||
|
.caseId(UUID.randomUUID().toString())
|
||||||
|
.title("案例 " + i)
|
||||||
|
.rootCause("原因")
|
||||||
|
.solution("方案")
|
||||||
|
.referenceCount(i * 10)
|
||||||
|
.build();
|
||||||
|
repository.save(caseLib);
|
||||||
|
}
|
||||||
|
|
||||||
|
List<CaseLibrary> topCases = repository.findTop10ByOrderByReferenceCountDesc();
|
||||||
|
assertFalse(topCases.isEmpty());
|
||||||
|
|
||||||
|
// 验证排序（第一个引用次数应该最高）
|
||||||
|
if (topCases.size() >= 2) {
|
||||||
|
assertTrue(topCases.get(0).getReferenceCount() >= topCases.get(1).getReferenceCount());
|
||||||
|
}
|
||||||
|
System.out.println("✓ 查询热门案例成功，返回 " + topCases.size() + " 条");
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,244 @@
|
|||||||
|
package org.example.service.session;
|
||||||
|
|
||||||
|
import org.example.domain.model.SessionContext;
|
||||||
|
import org.example.domain.model.ToolCall;
|
||||||
|
import org.junit.jupiter.api.BeforeEach;
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
import org.springframework.beans.factory.annotation.Autowired;
|
||||||
|
import org.springframework.boot.test.context.SpringBootTest;
|
||||||
|
import org.springframework.test.context.TestPropertySource;
|
||||||
|
|
||||||
|
import java.time.LocalDateTime;
|
||||||
|
import java.util.HashMap;
|
||||||
|
import java.util.Map;
|
||||||
|
import java.util.Optional;
|
||||||
|
import java.util.UUID;
|
||||||
|
|
||||||
|
import static org.junit.jupiter.api.Assertions.*;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* RedisSessionManager 单元测试
|
||||||
|
*/
|
||||||
|
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.NONE)
|
||||||
|
@TestPropertySource(properties = {
|
||||||
|
"spring.redis.host=119.29.78.52",
|
||||||
|
"spring.redis.port=6379",
|
||||||
|
"spring.redis.password=!Fucker123.."
|
||||||
|
})
|
||||||
|
class RedisSessionManagerTest {
|
||||||
|
|
||||||
|
@Autowired
|
||||||
|
private SessionManager sessionManager;
|
||||||
|
|
||||||
|
private String testSessionId;
|
||||||
|
|
||||||
|
@BeforeEach
|
||||||
|
void setUp() {
|
||||||
|
testSessionId = "test-session-" + UUID.randomUUID().toString();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testCreateAndGetSession() {
|
||||||
|
// 创建会话
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-123")
|
||||||
|
.businessId("order-456")
|
||||||
|
.traceId("trace-789")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
String sessionId = sessionManager.createSession(context, 300); // 5分钟
|
||||||
|
assertNotNull(sessionId);
|
||||||
|
assertEquals(testSessionId, sessionId);
|
||||||
|
System.out.println("✓ 创建会话成功: " + sessionId);
|
||||||
|
|
||||||
|
// 获取会话
|
||||||
|
Optional<SessionContext> retrieved = sessionManager.getSession(testSessionId);
|
||||||
|
assertTrue(retrieved.isPresent());
|
||||||
|
assertEquals("user-123", retrieved.get().getUserId());
|
||||||
|
assertEquals("ACTIVE", retrieved.get().getStatus());
|
||||||
|
assertNotNull(retrieved.get().getCreatedAt());
|
||||||
|
System.out.println("✓ 获取会话成功");
|
||||||
|
|
||||||
|
// 清理
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testUpdateSession() {
|
||||||
|
// 创建会话
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-update")
|
||||||
|
.status("ACTIVE")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.createSession(context, 300);
|
||||||
|
|
||||||
|
// 获取并更新
|
||||||
|
Optional<SessionContext> retrieved = sessionManager.getSession(testSessionId);
|
||||||
|
assertTrue(retrieved.isPresent());
|
||||||
|
|
||||||
|
SessionContext toUpdate = retrieved.get();
|
||||||
|
toUpdate.setStatus("COMPLETED");
|
||||||
|
toUpdate.setBusinessId("updated-business-id");
|
||||||
|
|
||||||
|
sessionManager.updateSession(toUpdate);
|
||||||
|
|
||||||
|
// 验证更新
|
||||||
|
Optional<SessionContext> updated = sessionManager.getSession(testSessionId);
|
||||||
|
assertTrue(updated.isPresent());
|
||||||
|
assertEquals("COMPLETED", updated.get().getStatus());
|
||||||
|
assertEquals("updated-business-id", updated.get().getBusinessId());
|
||||||
|
System.out.println("✓ 更新会话成功");
|
||||||
|
|
||||||
|
// 清理
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testDeleteSession() {
|
||||||
|
// 创建会话
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-delete")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.createSession(context, 300);
|
||||||
|
assertTrue(sessionManager.exists(testSessionId));
|
||||||
|
|
||||||
|
// 删除会话
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
assertFalse(sessionManager.exists(testSessionId));
|
||||||
|
System.out.println("✓ 删除会话成功");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testExists() {
|
||||||
|
assertFalse(sessionManager.exists(testSessionId));
|
||||||
|
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-exists")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.createSession(context, 300);
|
||||||
|
assertTrue(sessionManager.exists(testSessionId));
|
||||||
|
System.out.println("✓ 会话存在性检查成功");
|
||||||
|
|
||||||
|
// 清理
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testRefreshSession() {
|
||||||
|
// 创建会话
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-refresh")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.createSession(context, 60); // 1分钟
|
||||||
|
|
||||||
|
// 刷新过期时间
|
||||||
|
boolean refreshed = sessionManager.refreshSession(testSessionId, 600); // 延长到10分钟
|
||||||
|
assertTrue(refreshed);
|
||||||
|
assertTrue(sessionManager.exists(testSessionId));
|
||||||
|
System.out.println("✓ 刷新会话过期时间成功");
|
||||||
|
|
||||||
|
// 清理
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testAddToolCall() {
|
||||||
|
// 创建会话
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-toolcall")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.createSession(context, 300);
|
||||||
|
|
||||||
|
// 添加工具调用记录
|
||||||
|
Map<String, Object> args = new HashMap<>();
|
||||||
|
args.put("query", "test query");
|
||||||
|
args.put("limit", 10);
|
||||||
|
|
||||||
|
ToolCall toolCall = ToolCall.builder()
|
||||||
|
.toolName("search_documents")
|
||||||
|
.arguments(args)
|
||||||
|
.result("found 5 documents")
|
||||||
|
.status("SUCCESS")
|
||||||
|
.duration(150L)
|
||||||
|
.calledAt(LocalDateTime.now())
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.addToolCall(testSessionId, toolCall);
|
||||||
|
|
||||||
|
// 验证工具调用已添加
|
||||||
|
Optional<SessionContext> retrieved = sessionManager.getSession(testSessionId);
|
||||||
|
assertTrue(retrieved.isPresent());
|
||||||
|
assertFalse(retrieved.get().getToolCalls().isEmpty());
|
||||||
|
assertEquals(1, retrieved.get().getToolCalls().size());
|
||||||
|
assertEquals("search_documents", retrieved.get().getToolCalls().get(0).getToolName());
|
||||||
|
System.out.println("✓ 添加工具调用记录成功");
|
||||||
|
|
||||||
|
// 清理
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testUpdateStatus() {
|
||||||
|
// 创建会话
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-status")
|
||||||
|
.status("ACTIVE")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.createSession(context, 300);
|
||||||
|
|
||||||
|
// 更新状态
|
||||||
|
sessionManager.updateStatus(testSessionId, "COMPLETED");
|
||||||
|
|
||||||
|
// 验证状态已更新
|
||||||
|
Optional<SessionContext> retrieved = sessionManager.getSession(testSessionId);
|
||||||
|
assertTrue(retrieved.isPresent());
|
||||||
|
assertEquals("COMPLETED", retrieved.get().getStatus());
|
||||||
|
System.out.println("✓ 更新会话状态成功");
|
||||||
|
|
||||||
|
// 清理
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testMultipleToolCalls() {
|
||||||
|
// 创建会话
|
||||||
|
SessionContext context = SessionContext.builder()
|
||||||
|
.sessionId(testSessionId)
|
||||||
|
.userId("user-multi-tools")
|
||||||
|
.build();
|
||||||
|
|
||||||
|
sessionManager.createSession(context, 300);
|
||||||
|
|
||||||
|
// 添加多个工具调用
|
||||||
|
for (int i = 0; i < 3; i++) {
|
||||||
|
ToolCall toolCall = ToolCall.builder()
|
||||||
|
.toolName("tool_" + i)
|
||||||
|
.status("SUCCESS")
|
||||||
|
.calledAt(LocalDateTime.now())
|
||||||
|
.build();
|
||||||
|
sessionManager.addToolCall(testSessionId, toolCall);
|
||||||
|
}
|
||||||
|
|
||||||
|
// 验证所有工具调用
|
||||||
|
Optional<SessionContext> retrieved = sessionManager.getSession(testSessionId);
|
||||||
|
assertTrue(retrieved.isPresent());
|
||||||
|
assertEquals(3, retrieved.get().getToolCalls().size());
|
||||||
|
System.out.println("✓ 添加多个工具调用记录成功");
|
||||||
|
|
||||||
|
// 清理
|
||||||
|
sessionManager.deleteSession(testSessionId);
|
||||||
|
}
|
||||||
|
}
|
||||||
Reference in New Issue
Block a user