abapgit-agent 1.2.0 → 1.3.0

package/API.md

@@ -10,6 +10,8 @@ The ABAP system exposes these endpoints via SICF handler: `sap/bc/z_abapgit_agen
 | POST | `/pull` | Pull and activate repository |
 | POST | `/inspect` | Inspect source file for issues |
 | POST | `/unit` | Execute unit tests (AUnit) |
+| POST | `/tree` | Display package hierarchy tree |
+| POST | `/view` | View ABAP object definitions |
 
 ## GET /health
 
@@ -222,6 +224,223 @@ The endpoint parses file names to extract `obj_type` and `obj_name`, then runs A
 }
 ```
 
+## POST /tree
+
+Display package hierarchy tree from ABAP system.
+
+### Request Body
+
+```json
+{
+  "package": "$MY_PACKAGE",
+  "depth": 3,
+  "include_objects": true
+}
+```
+
+| Field | Type | Description |
+|-------|------|-------------|
+| `package` | String | Package name (required) |
+| `depth` | Integer | Maximum depth (default: 3, max: 10) |
+| `include_objects` | Boolean | Include object counts by type |
+
+### Response (success)
+
+```json
+{
+  "success": true,
+  "command": "TREE",
+  "package": "$MY_PACKAGE",
+  "message": "Tree retrieved successfully",
+  "parent_package": "$ZSAP_BASE",
+  "nodes": [
+    {
+      "package": "$MY_PACKAGE",
+      "parent": "",
+      "description": "$MY_PACKAGE",
+      "depth": 0,
+      "object_count": 10
+    },
+    {
+      "package": "$MY_SUBPACKAGE",
+      "parent": "$MY_PACKAGE",
+      "description": "$MY_SUBPACKAGE",
+      "depth": 1,
+      "object_count": 5
+    }
+  ],
+  "total_packages": 2,
+  "total_objects": 15,
+  "objects": [
+    { "object": "CLAS", "count": 8 },
+    { "object": "INTF", "count": 2 },
+    { "object": "TABL", "count": 5 }
+  ],
+  "error": ""
+}
+```
+
+### Response (error)
+
+```json
+{
+  "success": false,
+  "command": "TREE",
+  "package": "$NONEXISTENT",
+  "error": "Package $NONEXISTENT does not exist"
+}
+```
+
+## POST /view
+
+View ABAP object definitions directly from ABAP system.
+
+### Request Body
+
+```json
+{
+  "objects": ["ZCL_MY_CLASS", "ZIF_MY_INTERFACE", "SFLIGHT"],
+  "type": "CLAS"
+}
+```
+
+| Field | Type | Description |
+|-------|------|-------------|
+| `objects` | Array | List of object names (required) |
+| `type` | String | Object type (CLAS, INTF, TABL, STRU, DTEL). Auto-detected if not specified |
+
+### Supported Object Types
+
+| Type | Description |
+|------|-------------|
+| CLAS | Global ABAP class |
+| INTF | Global interface |
+| TABL | Database table |
+| STRU | Structure type |
+| DTEL | Data element |
+
+### Response (success - class/interface)
+
+```json
+{
+  "success": true,
+  "command": "VIEW",
+  "message": "Retrieved object(s)",
+  "objects": [
+    {
+      "name": "ZCL_MY_CLASS",
+      "type": "CLAS",
+      "type_text": "Class",
+      "description": "Class ZCL_MY_CLASS in $PACKAGE",
+      "source": "CLASS zcl_my_class DEFINITION PUBLIC.\n  PUBLIC SECTION.\n  ...",
+      "not_found": false,
+      "components": []
+    }
+  ],
+  "summary": {
+    "total": 1,
+    "by_type": ["CLAS"]
+  },
+  "error": ""
+}
+```
+
+### Response (success - table)
+
+```json
+{
+  "success": true,
+  "command": "VIEW",
+  "message": "Retrieved object(s)",
+  "objects": [
+    {
+      "name": "SFLIGHT",
+      "type": "TABL",
+      "type_text": "Table",
+      "description": "Table SFLIGHT in SAPBC_DATAMODEL",
+      "source": "",
+      "not_found": false,
+      "components": [
+        {
+          "field": "MANDT",
+          "key": true,
+          "type": "CLNT",
+          "length": 3,
+          "dataelement": "MANDT",
+          "description": "Client"
+        },
+        {
+          "field": "CARRID",
+          "key": true,
+          "type": "CHAR",
+          "length": 3,
+          "dataelement": "S_CARR_ID",
+          "description": "Airline Code"
+        }
+      ]
+    }
+  ],
+  "summary": {
+    "total": 1,
+    "by_type": ["TABL"]
+  },
+  "error": ""
+}
+```
+
+### Response (success - data element)
+
+```json
+{
+  "success": true,
+  "command": "VIEW",
+  "message": "Retrieved object(s)",
+  "objects": [
+    {
+      "name": "S_CARR_ID",
+      "type": "DTEL",
+      "type_text": "Data Element",
+      "description": "Airline Code",
+      "domain": "S_CARR_ID",
+      "domain_type": "CHAR",
+      "domain_length": 3,
+      "domain_decimals": 0,
+      "not_found": false,
+      "components": []
+    }
+  ],
+  "summary": {
+    "total": 1,
+    "by_type": ["DTEL"]
+  },
+  "error": ""
+}
+```
+
+### Response (not found)
+
+```json
+{
+  "success": true,
+  "command": "VIEW",
+  "message": "Retrieved object(s)",
+  "objects": [
+    {
+      "name": "ZIF_NONEXISTENT",
+      "type": "",
+      "type_text": "Unknown",
+      "not_found": true,
+      "components": []
+    }
+  ],
+  "summary": {
+    "total": 1,
+    "by_type": [""]
+  },
+  "error": ""
+}
+```
+
 ## Response Structure
 
 ### Pull Response Fields
@@ -261,6 +480,48 @@ The endpoint parses file names to extract `obj_type` and `obj_name`, then runs A
 | `message` | String | Status message |
 | `errors` | Array | Failed test details (empty if all tests pass) |
 
+### Tree Response Fields
+
+| Field | Type | Description |
+|-------|------|-------------|
+| `success` | Boolean | Whether the request succeeded |
+| `command` | String | Command name ("TREE") |
+| `package` | String | Root package name |
+| `message` | String | Status message |
+| `parent_package` | String | Parent package (empty if root) |
+| `nodes` | Array | Flat list of all packages |
+| `total_packages` | Integer | Total packages in tree |
+| `total_objects` | Integer | Total objects in tree |
+| `objects` | Array | Object counts by type |
+| `error` | String | Error message (empty if success) |
+
+### View Response Fields
+
+| Field | Type | Description |
+|-------|------|-------------|
+| `success` | Boolean | Whether the request succeeded |
+| `command` | String | Command name ("VIEW") |
+| `message` | String | Status message |
+| `objects` | Array | List of object information |
+| `summary` | Object | Summary with total and by_type |
+| `error` | String | Error message (empty if success) |
+
+### Object Fields (for View)
+
+| Field | Type | Description |
+|-------|------|-------------|
+| `name` | String | Object name |
+| `type` | String | Object type (CLAS, INTF, TABL, STRU, DTEL) |
+| `type_text` | String | Human-readable type |
+| `description` | String | Object description |
+| `source` | String | Source code (CLAS/INTF) |
+| `domain` | String | Domain name (DTEL) |
+| `domain_type` | String | Domain data type (DTEL) |
+| `domain_length` | Integer | Domain length (DTEL) |
+| `domain_decimals` | Integer | Domain decimals (DTEL) |
+| `not_found` | Boolean | true if object does not exist |
+| `components` | Array | Fields/components (TABL/STRU) |
+
 ### Error Item Fields
 
 | Field | Type | Description |

package/CLAUDE.md

@@ -181,6 +181,75 @@ abapgit-agent health
 }
 ```
 
+## Inspect Command
+
+### Description
+Run syntax check for ABAP objects. Supports both regular ABAP objects (classes, interfaces, programs) and CDS views (DDLS).
+
+### Usage
+```bash
+# Inspect single file
+abapgit-agent inspect --files abap/zcl_my_class.clas.abap
+
+# Inspect multiple files
+abapgit-agent inspect --files abap/zcl_class1.clas.abap,abap/zcl_class2.clas.abap
+
+# Inspect CDS view
+abapgit-agent inspect --files abap/zc_my_view.ddls.asddls
+
+# Inspect mixed file types (DDLS + CLAS)
+abapgit-agent inspect --files abap/zc_my_view.ddls.asddls,abap/zcl_my_class.clas.abap
+```
+
+### Supported Object Types
+
+| Type | Description | Validation Method |
+|------|-------------|------------------|
+| CLAS | Class | Code Inspector (SCI) |
+| INTF | Interface | Code Inspector (SCI) |
+| PROG | Program | Code Inspector (SCI) |
+| FUGR | Function Group | Code Inspector (SCI) |
+| DDLS | CDS View/Entity | DDL Handler (CL_DD_DDL_HANDLER_FACTORY) |
+
+### Output
+
+**Passed:**
+```
+✅ CLAS ZCL_MY_CLASS - Syntax check passed
+```
+
+**With Warnings:**
+```
+⚠️  DDLS ZC_MY_VIEW - Syntax check passed with warnings (4):
+
+Warnings:
+────────────────────────────────────────────────────────────
+  Line 9 : ParentPackage
+  Line 11 : SoftwareComponent
+```
+
+**Failed:**
+```
+❌ DDLS ZC_MY_VIEW - Syntax check failed (1 error(s)):
+
+Errors:
+────────────────────────────────────────────────────────────
+  Line 21, Column 12: Error message text
+```
+
+### Key Behaviors
+
+1. **Multiple files in one request** - All files are sent in a single API call for better performance
+2. **CDS View validation** - Uses `CL_DD_DDL_HANDLER_FACTORY` to validate CDS views
+3. **Check inactive version first** - For CDS views, checks the inactive version first (`get_state = 'M'`), then falls back to active version
+4. **Detailed error messages** - Uses `get_errors()` and `get_warnings()` methods from the exception to get detailed information
+5. **Per-object results** - Returns results for each object individually
+
+### File Format
+Files are parsed to extract `(obj_type, obj_name)`:
+- `zcl_my_class.clas.abap` → CLAS, ZCL_MY_CLASS
+- `zc_my_view.ddls.asddls` → DDLS, ZC_MY_VIEW
+
 ## Unit Command
 
 ### Description
@@ -371,6 +440,9 @@ abapgit-agent view --objects ZMY_STRUCT --type STRU
 # View data element type information
 abapgit-agent view --objects ZMY_DTEL --type DTEL
 
+# View CDS view definition
+abapgit-agent view --objects ZC_MY_CDS_VIEW --type DDLS
+
 # View class interface and methods
 abapgit-agent view --objects ZCL_UNKNOWN_CLASS
 
@@ -378,6 +450,24 @@ abapgit-agent view --objects ZCL_UNKNOWN_CLASS
 abapgit-agent view --objects ZIF_UNKNOWN_INTERFACE
 ```
 
+### When to Use View Command
+
+AI assistant SHOULD call `view` command when:
+
+- User asks to "check", "look up", or "explore" an unfamiliar object
+- Working with a table/structure and you don't know the field names/types
+- Calling a class/interface method and you don't know the parameters
+- User provides an object name that may not exist in the git repository
+- You need to verify an object exists before using it
+
+**Example workflow:**
+```
+User: "Check if SFLIGHT table has a PRICE field"
+
+Assistant: <calls `abapgit-agent view --objects SFLIGHT --type TABL`>
+→ Shows table structure with all fields including PRICE
+```
+
 ### Usage
 ```bash
 # View single object (auto-detect type from TADIR)
@@ -401,29 +491,37 @@ abapgit-agent view --objects ZCL_MY_CLASS --json
 | Parameter | Required | Description |
 |-----------|----------|-------------|
 | `--objects` | Yes | Comma-separated list of object names (e.g., `ZCL_MY_CLASS,ZIF_MY_INTERFACE`) |
-| `--type` | No | Object type for all objects (CLAS, INTF, TABL, STRU, DTEL, FUGR, PROG). Auto-detected from name prefix if not specified |
+| `--type` | No | Object type (CLAS, INTF, TABL, STRU, DTEL, TTYP, DDLS). Auto-detected from TADIR if not specified |
 | `--json` | No | Output raw JSON only (for scripting) |
 
 ### Supported Object Types
 
-| Type | Description | Detection Pattern |
-|------|-------------|-------------------|
-| CLAS | Class | ZCL_* |
-| INTF | Interface | ZIF_* |
-| TABL | Table | Z* (from DD02L) |
-| STRU | Structure | Z* (from DD02L) |
-| DTEL | Data Element | ZTY_*, ZS_* |
+| Type | Description |
+|------|-------------|
+| CLAS | Class |
+| INTF | Interface |
+| TABL | Table |
+| STRU | Structure |
+| DTEL | Data Element |
+| TTYP | Table Type |
+| DDLS | CDS View/Entity |
+
+**Note:** Object type is automatically detected from TADIR. Use `--type` only when you know the type and want to override auto-detection.
 
 ### Output
 
 **Human-readable:**
 ```
 📖 ZCL_MY_CLASS (Class)
-   My Custom Configuration Class
-   Methods: 3
-     - PUBLIC CONSTRUCTOR
-     - PUBLIC GET_CONFIG
-     - PUBLIC SET_CONFIG
+   Class ZCL_MY_CLASS in $PACKAGE
+
+CLASS zcl_my_class DEFINITION PUBLIC.
+
+  PUBLIC SECTION.
+    INTERFACES: zif_my_interface.
+    METHODS: constructor,
+      get_value RETURNING VALUE(rv_result) TYPE string.
+ENDCLASS.
 ```
 
 **JSON Output:**
@@ -431,20 +529,49 @@ abapgit-agent view --objects ZCL_MY_CLASS --json
 {
   "success": true,
   "command": "VIEW",
-  "message": "Retrieved 1 object(s)",
+  "message": "Retrieved object(s)",
   "objects": [
     {
       "name": "ZCL_MY_CLASS",
       "type": "CLAS",
       "type_text": "Class",
-      "description": "My Custom Configuration Class",
-      "methods": [...]
+      "description": "Class ZCL_MY_CLASS in $PACKAGE",
+      "source": "CLASS zcl_my_class DEFINITION PUBLIC...",
+      "not_found": false
     }
   ],
   "summary": { "total": 1 }
 }
 ```
 
+**Table Type Output:**
+```
+📖 ZMY_TTYP (Table Type)
+   Table Type ZMY_TTYP in $PACKAGE
+
+   Line Type: ZMY_STRUCTURE
+   Access Mode: STANDARD
+   Key Definition: WITH KEY
+```
+
+**CDS View Output:**
+```
+📖 ZC_MY_CDS_VIEW (CDS View)
+   CDS View ZC_MY_CDS_VIEW in $PACKAGE
+
+@AbapCatalog.sqlViewName: 'ZCMYVIEW'
+@AbapCatalog.compiler.compareFilter: true
+@AccessControl.authorizationCheck: #NOT_REQUIRED
+@EndUserText.label: 'My CDS View'
+define view ZC_MY_CDS_VIEW as select from tdevc
+{
+  key devclass as Devclass,
+      parentcl as ParentPackage,
+      ctext    as Description
+}
+where devclass not like '$%'
+```
+
 ### Error Handling
 
 | Error | Message |
@@ -519,6 +646,11 @@ export GIT_PASSWORD="git-token"
 
 ### Exploring Unknown ABAP Objects
 
+**Check package structure:**
+```bash
+abapgit-agent tree --package $MY_PACKAGE
+```
+
 **Before working with an unfamiliar table, structure, class, or interface:**
 
 ```bash
@@ -573,6 +705,10 @@ For quick ABAP code changes:
 4. Verify activation results
 5. Repeat until done
 
+## Creating CDS Views
+
+For guidelines on creating CDS views and CDS view entities, see **ABAP Code Generation** below.
+
 ## For ABAP Code Generation
 
 **NOTE**: This file is for developing the CLI tool itself. For guidelines on **generating ABAP code** for abapGit repositories, see `/abap/CLAUDE.md`. Copy that file to your ABAP repository root when setting up new projects.

package/RELEASE_NOTES.md

@@ -1,5 +1,18 @@
 # Release Notes
 
+## v1.3.0
+
+### New Features
+
+- **view Command**: Now supports TTYP (Table Type) and DDLS (CDS View)
+
+### Bug Fixes
+
+- Fixed CDS view source display in view command
+- Fixed interface exception handling
+
+---
+
 ## v1.2.0
 
 ### New Features