npx skills add ...
npx skills add arc-mcp/arc-1 --skill analyze-chat-session
Analyze the current conversation's tool calls, responses, errors, and approach to produce a structured feedback report for improving MCP tool usage with ARC-1. Use when the user asks to "analyze this session", "review my chat", "give feedback on tool usage", "how did I do", or wants a retrospective on a debugging session.
npx skills add arc-mcp/arc-1 --skill analyze-chat-session
Analyze the current conversation's tool calls, responses, errors, and overall approach to produce a structured feedback report. The goal is continuous improvement of MCP tool usage patterns, prompt strategies, and ARC-1 server capabilities.
This skill is introspective — the LLM analyzes its own chat session (the conversation you're currently in, or a referenced one) and generates actionable feedback.
Before using this skill's output outside the current system (e.g., pasting into a GitHub issue, sharing in a chat), the user MUST review it for sensitive data.
The analysis MUST NOT contain:
The LLM must actively redact these from the output, replacing them with generic placeholders like <SAP_HOST>, <USERNAME>, <OBJECT_NAME>, <TABLE_DATA_REDACTED>, etc.
After generating the report, remind the user: "Please review this report for any remaining sensitive information before sharing it outside your organization."
| Setting | Default | Rationale |
|---|---|---|
| Output format | issue (GitHub issue-ready) | Most actionable format |
| Severity filter | All levels | Don't miss anything |
| Focus area | Entire session | Comprehensive analysis |
| Sensitive data | Redact all (see Privacy Notice) | Safe by default |
Proceed immediately with Smart Defaults when triggered. No questions needed.
The user triggers this skill at any point during (or after) a conversation. No additional input is required — the LLM analyzes the current chat context. Proceed immediately with Smart Defaults.
Optionally, the user may specify:
issue (GitHub issue-ready), internal (detailed, for internal review), or summary (brief, 1 paragraph)Walk through the entire conversation and extract a structured catalog. For each turn, note:
What was the user trying to accomplish? Categorize it:
| Goal Category | Examples |
|---|---|
| Read / Explore | Read source code, understand a class, explore dependencies |
| Search / Discover | Find objects by name, search source code, locate implementations |
| Write / Modify | Create objects, update source, edit methods, batch operations |
| Diagnose / Debug | Syntax check, unit tests, ATC, short dumps, traces |
| Navigate / Understand | Find definition, find references, code completion |
| Query / Data | Table preview, freestyle SQL, data inspection |
| Context / Architecture | Dependency graphs, usage analysis, API contracts |
| Activate / Deploy | Activate objects, manage transports |
| Lint / Quality | Run linter, fix lint issues, check rules |
For each MCP tool call in the session, record:
| Field | What to capture |
|---|---|
| Tool | Which of the 11 tools was called (SAPRead, SAPSearch, etc.) |
| Operation | The specific action/type/include (e.g., SAPRead type=CLAS format=structured) |
| Intent | What information was the LLM trying to get? |
| Result | success, error, partial, empty, redundant |
| Error type | If failed: not_found, auth, safety, network, validation, btp_unsupported, session, other |
| Tokens | Rough estimate: was the response small (< 1KB), medium (1-10KB), or large (> 10KB)? |
| Necessary? | Was this call necessary, or could the info have been obtained differently? |
Look for these anti-patterns:
| Pattern | Description | Impact |
|---|---|---|
| Shotgun search | Multiple broad searches before narrowing | Wasted tokens and time |
| Missing SAPContext | Reading dependencies one-by-one instead of using SAPContext | 5-10x more calls than needed |
| Unstructured class read | Using plain SAPRead for a class instead of format=structured | Misses metadata, requires extra calls for includes |
| Retry without change | Repeating the same failing call without modifying parameters | Pure waste |
| Wrong type guess | Using wrong object type, then searching to find the right one | Could use SAPSearch first |
| Ignored error hints | Error response contains a hint (e.g., "use SAPSearch") but LLM didn't follow it | Missed recovery path |
| Redundant reads | Reading the same object multiple times in one session | Could cache/reuse from earlier response |
| Over-fetching | Reading entire objects when only a method or section was needed | Wasted tokens (use method param or edit_method) |
| Missing method listing | Modifying a class without first getting method="*" to understand the API | Blind modifications |
| Wrong action sequence | e.g., trying to activate before saving, or writing without reading first | Procedural error |
| Ignored BTP constraints | Attempting on-prem-only operations on BTP system | Known limitation, should check SAPManage features first |
Identify and highlight things that worked well:
format=structured, include=elements, method="*", etc.edit_method instead of full class rewriteIdentify approaches that eventually succeeded but were inefficient:
Identify approaches that never worked:
Suggestions the user can feed back into system prompts or skill instructions:
Suggestions that should become GitHub issues for the ARC-1 project:
| Category | Example |
|---|---|
| Missing operation | "No way to read message class text by message number" |
| Poor error message | "404 on FUNC doesn't suggest using SAPSearch to find the function group" |
| Missing parameter | "SAPRead for DDLS should support reading specific annotations" |
| Performance | "SAPContext depth=2 timed out for large dependency graphs" |
| BTP gap | "Cannot read service binding details on BTP" |
| Documentation | "Tool description doesn't mention that method='*' lists all methods" |
| Safety false positive | "Legitimate read operation blocked by safety check" |
| Caching opportunity | "Same object read 4 times — server should hint about SAPContext caching" |
Suggestions for the user's workflow:
For every recommendation, classify delivery scope explicitly:
| Bucket | Definition |
|---|---|
| Quick Win (This Session) | Low-risk prompt/skill/doc tweak that can be implemented immediately without new backend primitives |
| Needs Plan + Implementation Phase | Requires code changes, new ADT integration, schema/tool contract updates, or live SAP regression testing |
Output two lists:
This triage section is mandatory in issue and internal outputs.
issue (Default — GitHub Issue Ready)Generate a report structured for creating a GitHub issue. Use this template:
internal (Detailed Internal Review)Same as issue but includes:
summary (Brief)A single paragraph summarizing: goal, outcome, top 2 things that worked, top 2 things that didn't, and the single most impactful improvement suggestion.
After presenting the report, offer:
.claude/commands/*.md fileUse this reference to assess whether the LLM used the right tools. This is a concise mapping of common goals to optimal tool usage.
| Object Type | Key Parameters | Notes |
|---|---|---|
| CLAS | format=structured (metadata+all includes), method="*" (list methods), method="name" (single method source), include=testclasses|definitions|implementations|macros | Always prefer format=structured for initial read |
| DDLS | include=elements (field listing) | Field list is essential for understanding CDS views |
| FUNC | group=<FUGR> (required if not cached), expand_includes=true (full group source) | Group resolution is a common failure point |
| SRVB | Returns parsed binding info | Not raw DDL |
| TABLE_CONTENTS | maxRows, sqlFilter | Data preview — subject to blockData safety |
searchType=object — by name/wildcardsearchType=source_code — full-text (may be unavailable on some systems)SAPWrite action=create or action=update or action=edit_methodSAPActivate (single or batch with objects[])SAPDiagnose action=syntax or action=unittest to verifySAPContext action=deps — dependency graph (preferred over manual reads)SAPContext action=usages — live reverse dependencies from SAPSAPNavigate action=definition — go-to-definition (needs URI + line/col + source)SAPNavigate action=references — where-used| Error | Recovery |
|---|---|
| 404 Not Found | → SAPSearch to verify name/type |
| FUNC without group | → SAPSearch for function name, extract group from result |
| BTP type not supported | → Check BTP_HINTS in error, use equivalent cloud type |
| Source search unavailable | → Use SAPQuery or object search instead |
| SAPContext usages fails | → Use SAPNavigate references as fallback |
| Activation fails | → Read activation error, fix source, retry |
| Lock conflict | → Object locked by another user, wait or use different object |
| Situation | How to Handle |
|---|---|
| Very short session (< 3 tool calls) | Produce a brief summary, note that limited data makes analysis less useful |
| No tool calls in session | Report that no MCP tool usage was detected; suggest this skill is meant for sessions with SAP operations |
| All calls succeeded | Focus the report on efficiency and optimization opportunities |
| All calls failed | Focus on root cause analysis and whether the goal was achievable with current capabilities |
| Mixed results | Full analysis with emphasis on the failure→recovery patterns |
| Session used non-ARC-1 tools | Catalog those separately; focus ARC-1 analysis on the SAP-related calls |