abapgit-agent 1.1.6 → 1.3.0

package/CLAUDE.md

@@ -38,6 +38,7 @@ ABAP REST Handler (ZCL_ABGAGT_REST_HANDLER)
 Resource: ZCL_ABGAGT_RESOURCE_PULL → ZCL_ABGAGT_CMD_FACTORY → ZCL_ABGAGT_COMMAND_PULL → ZCL_ABGAGT_AGENT
 Resource: ZCL_ABGAGT_RESOURCE_INSPECT → ZCL_ABGAGT_CMD_FACTORY → ZCL_ABGAGT_COMMAND_INSPECT → ZCL_ABGAGT_AGENT
 Resource: ZCL_ABGAGT_RESOURCE_UNIT → ZCL_ABGAGT_CMD_FACTORY → ZCL_ABGAGT_COMMAND_UNIT → ZCL_ABGAGT_AGENT
+Resource: ZCL_ABGAGT_RESOURCE_TREE → ZCL_ABGAGT_CMD_FACTORY → ZCL_ABGAGT_COMMAND_TREE → ZCL_ABGAGT_AGENT
 ```
 
 ### ABAP Objects
@@ -49,6 +50,7 @@ Resource: ZCL_ABGAGT_RESOURCE_UNIT → ZCL_ABGAGT_CMD_FACTORY → ZCL_ABGAGT_COM
 | `ZCL_ABGAGT_COMMAND_PULL` | Pull command - implements ZIF_ABGAGT_COMMAND |
 | `ZCL_ABGAGT_COMMAND_INSPECT` | Inspect command - implements ZIF_ABGAGT_COMMAND |
 | `ZCL_ABGAGT_COMMAND_UNIT` | Unit command - implements ZIF_ABGAGT_COMMAND |
+| `ZCL_ABGAGT_COMMAND_TREE` | Tree command - displays package hierarchy |
 | `ZIF_ABGAGT_COMMAND` | Command interface with constants |
 | `ZIF_ABGAGT_CMD_FACTORY` | Factory interface |
 
@@ -73,6 +75,14 @@ abapgit-agent inspect --files abap/zcl_my_class.clas.abap
 # Run unit tests for ABAP test classes
 abapgit-agent unit --files abap/zcl_my_test.clas.testclasses.abap
 
+# Display package hierarchy tree
+abapgit-agent tree --package $MY_PACKAGE
+
+# View ABAP object definitions from ABAP system
+abapgit-agent view --objects ZCL_MY_CLASS
+abapgit-agent view --objects ZIF_MY_INTERFACE --type INTF
+abapgit-agent view --objects ZCL_CLASS1,ZCL_CLASS2 --json
+
 # Health check
 abapgit-agent health
 
@@ -85,6 +95,22 @@ abapgit-agent status
 ### Description
 Pull and activate ABAP objects from git repository.
 
+### Rule: Pull All Changed Files Together
+
+**CRITICAL**: When multiple ABAP files are changed, pull them ALL together in a single command:
+
+```bash
+# WRONG - Pull files one by one (may cause activation issues)
+abapgit-agent pull --files zcl_class1.clas.abap
+abapgit-agent pull --files zcl_class2.clas.abap
+abapgit-agent pull --files zif_interface1.intf.abap
+
+# CORRECT - Pull all changed files together
+abapgit-agent pull --files zcl_class1.clas.abap,zcl_class2.clas.abap,zif_interface1.intf.abap
+```
+
+**Why?** ABAP objects often have dependencies on each other. Pulling separately can cause activation errors if dependent objects haven't been activated yet. Pulling together ensures the ABAP system processes all changes atomically.
+
 ### Usage
 ```bash
 # Auto-detect git remote and branch from current directory
@@ -155,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
@@ -230,6 +325,260 @@ The unit command uses `CL_SUT_AUNIT_RUNNER` to execute AUnit tests:
 - `tab_objects` - Get detailed results with nested structure:
   - `TAB_TESTCLASSES` → `TAB_METHODS` → `STR_ERROR` → `STR_ERROR_CORE`
 
+## Tree Command
+
+### Description
+Display the package hierarchy tree from an ABAP system, showing parent packages, sub-packages, and object counts.
+
+### Usage
+```bash
+# Basic usage
+abapgit-agent tree --package $MY_PACKAGE
+
+# With object breakdown by type
+abapgit-agent tree --package $MY_PACKAGE --include-objects
+
+# Limit depth (default: 3, max: 10)
+abapgit-agent tree --package $MY_PACKAGE --depth 2
+
+# JSON output for scripting
+abapgit-agent tree --package $MY_PACKAGE --json
+```
+
+### Parameters
+
+| Parameter | Required | Description |
+|-----------|----------|-------------|
+| `--package` | Yes | ABAP package name (e.g., `$ZMY_PACKAGE`, `ZMY_PACKAGE`) |
+| `--depth` | No | Maximum depth to traverse (default: 3, max: 10) |
+| `--include-objects` | No | Include object counts breakdown by type |
+| `--json` | No | Output raw JSON only (for scripting) |
+
+### Output
+
+**Human-readable with AI metadata:**
+```
+🌳 Package Tree: $ZMAIN_PACKAGE
+
+📦 $ZMAIN_PACKAGE (Main Package)
+   ├─ 📦 $ZMAIN_SUB1 (Sub Package 1)
+   │    ├─ 📦 $ZMAIN_SUB1_A (Sub Package 1A)
+   │    └─ 📦 $ZMAIN_SUB1_B (Sub Package 1B)
+   └─ 📦 $ZMAIN_SUB2 (Sub Package 2)
+
+📊 Summary
+PACKAGES: 4
+OBJECTS: 127
+
+<!-- AI_METADATA_START -->
+{"package":"$ZMAIN_PACKAGE","parent":"$ZSAP_BASE","total_packages":4,"total_objects":127}
+<!-- AI_METADATA_END -->
+```
+
+**With object breakdown:**
+```
+📊 Summary
+PACKAGES: 4
+OBJECTS: 127
+TYPES: CLAS=10 INTF=2 PROG=11 FUGR=1 TABL=3
+```
+
+### JSON Output
+```json
+{
+  "SUCCESS": true,
+  "COMMAND": "TREE",
+  "PACKAGE": "$ZMAIN_PACKAGE",
+  "PARENT_PACKAGE": "$ZSAP_BASE",
+  "NODES": [
+    {
+      "PACKAGE": "$ZMAIN_PACKAGE",
+      "PARENT": "",
+      "DESCRIPTION": "$ZMAIN_PACKAGE",
+      "DEPTH": 0,
+      "OBJECT_COUNT": 11
+    }
+  ],
+  "TOTAL_PACKAGES": 4,
+  "TOTAL_OBJECTS": 127,
+  "OBJECTS": [
+    { "OBJECT": "CLAS", "COUNT": 10 },
+    { "OBJECT": "INTF", "COUNT": 2 },
+    { "OBJECT": "PROG", "COUNT": 11 },
+    { "OBJECT": "FUGR", "COUNT": 1 },
+    { "OBJECT": "TABL", "COUNT": 3 }
+  ],
+  "ERROR": ""
+}
+```
+
+### Error Handling
+
+| Error | Message |
+|-------|---------|
+| Package not found | `Package <name> does not exist in the system` |
+| Invalid package name | `Invalid package name: <name>` |
+| Access denied | `Access denied to package information` |
+| Depth too large | `Depth value too large (max: 10)` |
+
+## View Command
+
+### Description
+View ABAP object definitions directly from the ABAP system without pulling from git. **This is the PRIMARY way to explore unfamiliar ABAP objects** - tables, structures, classes, interfaces, and data elements.
+
+**When you encounter an unknown table or structure, use this command instead of guessing!**
+
+### Explore Unknown Tables/Structures
+
+```bash
+# View table fields - see all columns, keys, and descriptions
+abapgit-agent view --objects ZMY_TABLE --type TABL
+
+# View structure components
+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
+
+# View interface definition
+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)
+abapgit-agent view --objects ZCL_MY_CLASS
+abapgit-agent view --objects ZIF_MY_INTERFACE
+
+# View with explicit type
+abapgit-agent view --objects ZCL_MY_CLASS --type CLAS
+abapgit-agent view --objects ZIF_MY_INT --type INTF
+abapgit-agent view --objects ZMY_TABLE --type TABL
+
+# View multiple objects
+abapgit-agent view --objects ZCL_CLASS1,ZCL_CLASS2,ZIF_INTERFACE1
+
+# JSON output (for scripting/AI processing)
+abapgit-agent view --objects ZCL_MY_CLASS --json
+```
+
+### Parameters
+
+| Parameter | Required | Description |
+|-----------|----------|-------------|
+| `--objects` | Yes | Comma-separated list of object names (e.g., `ZCL_MY_CLASS,ZIF_MY_INTERFACE`) |
+| `--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 |
+|------|-------------|
+| 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)
+   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:**
+```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...",
+      "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 |
+|-------|---------|
+| Object not found | `Object <name> not found` |
+| Invalid object type | `Unsupported object type: <type>` |
+
 ## Status Check
 
 ### Description
@@ -295,6 +644,37 @@ export GIT_PASSWORD="git-token"
 
 ## Development Workflow
 
+### 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
+# Don't guess! Use view command to explore:
+
+# Check table structure
+abapgit-agent view --objects ZMY_TABLE --type TABL
+
+# Check structure components
+abapgit-agent view --objects ZMY_STRUCT --type STRU
+
+# Check class methods and interface
+abapgit-agent view --objects ZCL_UNKNOWN_CLASS
+
+# Check interface definition
+abapgit-agent view --objects ZIF_UNKNOWN_INTERFACE
+
+# Check data element type
+abapgit-agent view --objects ZMY_DTEL --type DTEL
+
+# JSON output for programmatic use
+abapgit-agent view --objects ZMY_TABLE --type TABL --json
+```
+
 ### CLI Tool Development
 
 1. Make changes to CLI code (JavaScript)
@@ -325,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/README.md

@@ -25,7 +25,7 @@ git clone https://github.com/user/abap-project.git
 cd abap-project
 
 # 3. Initialize configuration
-abapgit-agent init --folder /abap --package ZMY_PACKAGE
+abapgit-agent init --folder /abap/ --package ZMY_PACKAGE
 
 # 4. Edit .abapGitAgent with credentials
 
@@ -47,7 +47,7 @@ See [Creating New ABAP Projects](INSTALL.md#creating-new-abap-projects) to set u
 
 ```bash
 # Initialize local configuration for existing git repo
-abapgit-agent init --folder /abap --package ZMY_PACKAGE
+abapgit-agent init --folder /abap/ --package ZMY_PACKAGE
 
 # Create online repository in ABAP
 abapgit-agent create
@@ -94,6 +94,18 @@ abapgit-agent unit --files abap/zcl_my_test.clas.testclasses.abap
 abapgit-agent unit --files abap/zcl_test1.clas.testclasses.abap,abap/zcl_test2.clas.testclasses.abap
 ```
 
+### Explore Commands
+
+```bash
+# Display package hierarchy tree
+abapgit-agent tree --package $MY_PACKAGE
+
+# View object definitions from ABAP system (classes, interfaces, tables, data elements)
+abapgit-agent view --objects ZCL_MY_CLASS
+abapgit-agent view --objects SFLIGHT --type TABL
+abapgit-agent view --objects S_CARR_ID --type DTEL
+```
+
 ### Utility Commands
 
 ```bash
@@ -129,6 +141,8 @@ npm run pull -- --url <git-url> --branch main
 | pull Command | [docs/pull-command.md](docs/pull-command.md) |
 | inspect Command | [docs/inspect-command.md](docs/inspect-command.md) |
 | unit Command | [docs/unit-command.md](docs/unit-command.md) |
+| tree Command | [docs/tree-command.md](docs/tree-command.md) |
+| view Command | [docs/view-command.md](docs/view-command.md) |
 | REST API Reference | [API.md](API.md) |
 | Error Handling | [ERROR_HANDLING.md](ERROR_HANDLING.md) |
 | ABAP Coding Guidelines | [abap/CLAUDE.md](abap/CLAUDE.md) |

package/RELEASE_NOTES.md

@@ -1,14 +1,99 @@
-## What's New
+# 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
+
+#### tree Command
+Display package hierarchy tree from ABAP system:
+```bash
+# Basic usage
+abapgit-agent tree --package $MY_PACKAGE
+
+# With object counts
+abapgit-agent tree --package $MY_PACKAGE --include-objects
+
+# Limit depth
+abapgit-agent tree --package $MY_PACKAGE --depth 2
+
+# JSON output for scripting
+abapgit-agent tree --package $MY_PACKAGE --json
+```
+
+#### view Command
+View ABAP object definitions directly from ABAP system without pulling from git:
+
+```bash
+# View class definition
+abapgit-agent view --objects ZCL_MY_CLASS
+
+# View interface
+abapgit-agent view --objects ZIF_MY_INTERFACE
+
+# View table structure
+abapgit-agent view --objects SFLIGHT --type TABL
+
+# View data element
+abapgit-agent view --objects S_CARR_ID --type DTEL
+
+# View multiple objects
+abapgit-agent view --objects ZCL_CLASS1,ZIF_INTERFACE1,ZMY_TABLE
+
+# Auto-detect type from TADIR
+abapgit-agent view --objects SFLIGHT
+
+# JSON output for scripting
+abapgit-agent view --objects ZCL_MY_CLASS --json
+```
+
+**Supported Object Types:**
+- CLAS - Global ABAP class
+- INTF - Global interface
+- TABL - Database table
+- STRU - Structure type
+- DTEL - Data element
+
+### Improvements
+
+- **Auto-detection**: view command now queries TADIR to automatically detect object types (no need to specify `--type` for most objects)
+- **Lowercase support**: Object names and types are case-insensitive
+- **Error handling**: Non-existent objects now display clear error messages
+- **Cleaner output**: Removed AI metadata block from tree command output
+
+### Bug Fixes
+
+- Fixed non-existent object detection in view command
+- Fixed source code display for class/interface definitions
+- Fixed table and structure field display formatting
+- Fixed data element output with domain information
+
+### Documentation
+
+- Added comprehensive docs/tree-command.md
+- Added comprehensive docs/view-command.md
+- Updated README.md with tree and view command examples
+- Updated CLAUDE.md with view command usage for exploring ABAP objects
+
+---
+
+## v1.1.6
 
 - Added `create` command to create online repository in ABAP system
 - Added `import` command to import objects from ABAP package to git
 - Added unit tests for create and import commands
 - Added GitHub Actions workflow for automated npm releases
-- Improved .gitignore handling in init command (adds .abapGitAgent and cookies file)
+- Improved .gitignore handling in init command
 - Updated commands documentation
-
-## Installation
-
-```bash
-npm install -g abapgit-agent
-```