abapgit-agent 1.4.0 → 1.6.0

package/README.md

@@ -124,6 +124,7 @@ abapgit-agent health
 # Check integration status
 abapgit-agent status
 ```
+Note: All ABAP commands automatically check CLI/ABAP API version compatibility.
 
 ## Local Development
 
@@ -153,6 +154,7 @@ npm run pull -- --url <git-url> --branch main
 | tree Command | [docs/tree-command.md](docs/tree-command.md) |
 | view Command | [docs/view-command.md](docs/view-command.md) |
 | preview Command | [docs/preview-command.md](docs/preview-command.md) |
+| ref Command | [docs/ref-command.md](docs/ref-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/abap/guidelines/00_index.md

@@ -0,0 +1,35 @@
+# ABAP Coding Guidelines Index
+
+This folder contains detailed ABAP coding guidelines that can be searched using the `ref` command.
+
+## Guidelines Available
+
+| File | Topic |
+|------|-------|
+| `01_sql.md` | ABAP SQL Best Practices |
+| `02_exceptions.md` | Exception Handling |
+| `03_testing.md` | Unit Testing (including CDS) |
+| `04_cds.md` | CDS Views |
+| `05_classes.md` | ABAP Classes and Objects |
+| `06_objects.md` | Object Naming Conventions |
+| `07_json.md` | JSON Handling |
+| `08_abapgit.md` | abapGit XML Metadata Templates |
+
+## Usage
+
+These guidelines are automatically searched by the `ref` command:
+
+```bash
+# Search across all guidelines
+abapgit-agent ref "CORRESPONDING"
+
+# List all topics
+abapgit-agent ref --list-topics
+```
+
+## Adding Custom Guidelines
+
+To add your own guidelines:
+1. Create a new `.md` file in this folder
+2. Follow the naming convention: `XX_name.md`
+3. Export to reference folder: `abapgit-agent ref export`

package/abap/guidelines/01_sql.md

@@ -0,0 +1,72 @@
+# ABAP SQL Best Practices
+
+When writing ABAP SQL (Open SQL) queries, follow these rules:
+
+## 1. Host Variables - Use @ Prefix
+
+Use `@` prefix for host variables in ABAP SQL:
+
+```abap
+" Correct
+SELECT * FROM tadir WHERE devclass = @lv_package.
+
+" Wrong - no @ prefix
+SELECT * FROM tadir WHERE devclass = lv_package.
+```
+
+## 2. Range Tables for IN Clauses
+
+When filtering with `IN`, use a range table with `@` prefix:
+
+```abap
+" Define range table
+DATA lt_type_range TYPE RANGE OF tadir-object.
+ls_type-sign = 'I'.
+ls_type-option = 'EQ'.
+ls_type-low = 'CLAS'.
+APPEND ls_type TO lt_type_range.
+
+" Use with @ prefix
+SELECT object, obj_name FROM tadir
+  WHERE object IN @lt_type_range
+  INTO TABLE @lt_objects.
+```
+
+## 3. SELECT Statement Clause Order
+
+The correct sequence is:
+```
+SELECT → FROM → WHERE → ORDER BY → INTO → UP TO → OFFSET
+```
+
+```abap
+SELECT object, obj_name FROM tadir
+  WHERE devclass = @lv_package
+    AND object IN @lt_type_range
+  ORDER BY object, obj_name
+  INTO TABLE @lt_objects
+  UP TO @lv_limit ROWS
+  OFFSET @lv_offset.
+```
+
+## 4. Fixed Point Arithmetic (FIXPT)
+
+For numeric operations in ABAP SQL (especially with UP TO/OFFSET), enable FIXPT in the class XML:
+
+```xml
+<VSEOCLASS>
+  <FIXPT>X</FIXPT>
+</VSEOCLASS>
+```
+
+## 5. Field Separation
+
+Always separate fields with commas in SELECT:
+
+```abap
+" Correct
+SELECT object, obj_name FROM tadir ...
+
+" Wrong - missing comma
+SELECT object obj_name FROM tadir ...
+```

package/abap/guidelines/02_exceptions.md

@@ -0,0 +1,108 @@
+# Exception Handling - Classical vs Class-Based
+
+ABAP has two exception handling mechanisms. Using the wrong one causes silent failures.
+
+## Quick Identification
+
+| Method Signature | Exception Type | Handling Pattern |
+|-----------------|----------------|------------------|
+| `EXCEPTIONS exc = 1` | Classical | `sy-subrc` check |
+| `RAISING cx_...` | Class-based | `TRY-CATCH` |
+
+## Classical Exceptions (Old Style)
+
+Methods declared with `EXCEPTIONS` in signature:
+
+```abap
+" Method declaration
+class-methods describe_by_name
+  importing p_name type any
+  returning value(p_descr_ref) type ref to cl_abap_typedescr
+  exceptions
+    type_not_found.  " <- Classical!
+
+" Calling with classical exception
+cl_abap_structdescr=>describe_by_name(
+  EXPORTING
+    p_name = iv_tabname
+  RECEIVING
+    p_descr_ref = lo_descr
+  EXCEPTIONS
+    type_not_found = 1
+    OTHERS = 2 ).
+IF sy-subrc <> 0.
+  " Handle error
+ENDIF.
+```
+
+**Common classical exception methods:**
+- `cl_abap_structdescr=>describe_by_name` - `type_not_found`
+- `cl_abap_tabledescr=>describe_by_name` - `type_not_found`
+- Many RTTI (Run Time Type Information) methods
+
+## Class-Based Exceptions (Modern)
+
+Methods declared with `RAISING`:
+
+```abap
+" Method declaration
+methods get_ref
+  importing p_name type string
+  returning value(p_ref) type ref to cl_ci_checkvariant
+  raising cx_ci_checkvariant.  " <- Class-based!
+
+" Calling with TRY-CATCH
+TRY.
+    lo_variant = cl_ci_checkvariant=>get_ref( p_name = lv_variant ).
+  CATCH cx_ci_checkvariant INTO DATA(lx_error).
+    " Handle error
+ENDTRY.
+```
+
+## The Mistake to Avoid
+
+**WRONG**: Using TRY-CATCH for classical exceptions
+```abap
+" This does NOT catch type_not_found!
+TRY.
+    lo_descr = cl_abap_structdescr=>describe_by_name( iv_name ).
+  CATCH cx_root.  " Never reached!
+ENDTRY.
+```
+
+**CORRECT**: Use proper pattern for each type
+```abap
+" Classical - use sy-subrc
+cl_abap_structdescr=>describe_by_name(
+  EXPORTING p_name = iv_name
+  RECEIVING p_descr_ref = lo_descr
+  EXCEPTIONS type_not_found = 1 ).
+IF sy-subrc <> 0.
+  " Handle error
+ENDIF.
+
+" Class-based - use TRY-CATCH
+TRY.
+    lo_obj = cl_ci_checkvariant=>get_ref( lv_name ).
+  CATCH cx_root INTO DATA(lx_error).
+    " Handle error
+ENDTRY.
+```
+
+## How to Check Which to Use
+
+Use the `view` command to check method signature:
+
+```bash
+abapgit-agent view --objects CL_ABAP_STRUCTDESCR
+```
+
+Look for:
+- `exceptions TYPE_NOT_FOUND` → Classical (use `sy-subrc`)
+- `raising CX_SY_RTTI` → Class-based (use `TRY-CATCH`)
+
+Or search the cheat sheets:
+
+```bash
+abapgit-agent ref --topic exceptions
+```

package/abap/guidelines/03_testing.md

@@ -0,0 +1,252 @@
+# Unit Testing
+
+## Unit Testing with Local Test Classes
+
+### File Structure
+
+For ABAP local unit tests, use a **separate file** with `.testclasses.abap` extension:
+
+```
+abap/
+  zcl_my_class.clas.abap          <- Main class (no test code)
+  zcl_my_class.clas.testclasses.abap  <- Local test class
+  zcl_my_class.clas.xml           <- XML with WITH_UNIT_TESTS = X
+```
+
+### Required Elements
+
+1. **Test class file** (`zcl_my_class.clas.testclasses.abap`):
+   ```abap
+   *"* use this source file for your test class implementation
+   *"* local test class
+   CLASS ltcl_zcl_my_class DEFINITION FOR TESTING DURATION SHORT RISK LEVEL HARMLESS.
+     PRIVATE SECTION.
+       DATA mo_cut TYPE REF TO zcl_my_class.
+       METHODS setup.
+       METHODS test_method1 FOR TESTING.
+       METHODS test_method2 FOR TESTING.
+   ENDCLASS.
+
+   CLASS ltcl_zcl_my_class IMPLEMENTATION.
+     METHOD setup.
+       CREATE OBJECT mo_cut.
+     ENDMETHOD.
+     METHOD test_method1.
+       " Test code using cl_abap_unit_assert
+     ENDMETHOD.
+   ENDCLASS.
+   ```
+
+2. **XML metadata** (`zcl_my_class.clas.xml`):
+   ```xml
+   <VSEOCLASS>
+     ...
+     <WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>
+   </VSEOCLASS>
+   ```
+
+### Naming Conventions
+
+- Test class name: `LTCL_ZCL_<CLASSNAME>` (e.g., `LTCL_ZCL_COMMAND_PULL`)
+- Test methods: `TEST_<methodname> FOR TESTING` or simply `test_method FOR TESTING`
+- Test file: `<classname>.clas.testclasses.abap`
+
+### CRITICAL: Method Name Length Limit
+
+**Test method names MUST NOT exceed 30 characters!**
+
+```abap
+" WRONG - 34 characters (syntax error)
+METHODS test_execute_with_minimal_params FOR TESTING.
+
+" CORRECT - 18 characters
+METHODS test_exec_minimal FOR TESTING.
+```
+
+Examples of compliant names:
+- `test_get_name` (13 chars)
+- `test_exec_minimal` (18 chars)
+- `test_exec_files` (16 chars)
+- `test_interface` (15 chars)
+
+### Common Assertions
+
+```abap
+cl_abap_unit_assert=>assert_equals( act = lv_actual exp = lv_expected msg = 'Error message' ).
+cl_abap_unit_assert=>assert_not_initial( act = lv_data msg = 'Should not be initial' ).
+cl_abap_unit_assert=>assert_bound( act = lo_ref msg = 'Should be bound' ).
+cl_abap_unit_assert=>assert_true( act = lv_bool msg = 'Should be true' ).
+```
+
+### What NOT To Do
+
+- ❌ Don't add test methods directly in the main `.clas.abap` file
+- ❌ Don't use `CLASS ... DEFINITION ...` without the special comment header
+- ❌ Don't reference `<TESTCLASS>` in XML - abapGit auto-detects `.testclasses.abap`
+- ❌ Don't use nested local classes inside the main class definition
+
+### Running Tests
+
+In ABAP: SE24 → Test → Execute Unit Tests
+
+Or via abapGit: Pull the files and run tests in the ABAP system.
+
+## Unit Testing CDS Views
+
+When testing code that uses CDS view entities, you can use the **CDS Test Double Framework** (`CL_CDS_TEST_ENVIRONMENT`) to create test doubles for CDS views. This allows you to inject test data without affecting production data.
+
+### When to Use CDS Test Doubles
+
+- Testing code that reads from CDS views
+- Need controlled test data (not production data)
+- Testing CDS view logic with specific scenarios
+
+### CDS Test Double Framework
+
+Use `CL_CDS_TEST_ENVIRONMENT` for controlled test data:
+
+```abap
+"-------------------------
+" CLASS DEFINITION
+"-------------------------
+CLASS ltcl_cds_test DEFINITION FOR TESTING RISK LEVEL HARMLESS DURATION SHORT FINAL.
+
+  PRIVATE SECTION.
+    " IMPORTANT: Use interface type, not class type!
+    DATA mo_cds_env TYPE REF TO if_cds_test_environment.
+
+    " IMPORTANT: class_setup/teardown must be CLASS-METHODS (static)!
+    CLASS-DATA mo_cds_env_static TYPE REF TO if_cds_test_environment.
+
+    METHODS setup.
+    METHODS test_cds_with_doubles FOR TESTING.
+
+    CLASS-METHODS: class_setup,
+                   class_teardown.
+
+ENDCLASS.
+
+"-------------------------
+" CLASS IMPLEMENTATION
+"-------------------------
+CLASS ltcl_cds_test IMPLEMENTATION.
+
+  METHOD class_setup.
+    " Create CDS test environment - framework auto-creates doubles for dependencies
+    mo_cds_env_static = cl_cds_test_environment=>create(
+      i_for_entity = 'ZC_MY_CDS_VIEW' ).
+  ENDMETHOD.
+
+  METHOD class_teardown.
+    " Clean up test environment
+    mo_cds_env_static->destroy( ).
+  ENDMETHOD.
+
+  METHOD setup.
+    " IMPORTANT: Assign static env to instance and clear doubles
+    mo_cds_env = mo_cds_env_static.
+    mo_cds_env->clear_doubles( ).
+  ENDMETHOD.
+
+  METHOD test_cds_with_doubles.
+    " IMPORTANT: Must declare table type first, cannot inline in VALUE!
+    DATA lt_test_data TYPE TABLE OF zc_my_cds_view WITH EMPTY KEY.
+    lt_test_data = VALUE #(
+      ( field1 = 'A' field2 = 100 )
+      ( field1 = 'B' field2 = 200 ) ).
+
+    " Insert test data using named parameter
+    mo_cds_env->insert_test_data( i_data = lt_test_data ).
+
+    " Select from CDS view
+    SELECT * FROM zc_my_cds_view INTO TABLE @DATA(lt_result).
+
+    " Verify results
+    cl_abap_unit_assert=>assert_not_initial(
+      act = lt_result
+      msg = 'Result should not be empty' ).
+
+    cl_abap_unit_assert=>assert_equals(
+      act = lines( lt_result )
+      exp = 2
+      msg = 'Expected 2 rows' ).
+  ENDMETHOD.
+
+ENDCLASS.
+```
+
+### Testing CDS Views with Aggregations (SUM, COUNT, GROUP BY)
+
+For CDS views with aggregations, insert test data into the **base tables** (SFLIGHT, SCARR, SBOOK), not directly into the CDS view:
+
+```abap
+METHOD test_aggregation.
+  " Insert data into base tables via CDS test doubles
+  DATA lt_scarr TYPE TABLE OF scarr WITH EMPTY KEY.
+  lt_scarr = VALUE #( ( carrid = 'LH' carrname = 'Lufthansa' currcode = 'EUR' ) ).
+  mo_cds_env->insert_test_data( i_data = lt_scarr ).
+
+  DATA lt_sflight TYPE TABLE OF sflight WITH EMPTY KEY.
+  lt_sflight = VALUE #( ( carrid = 'LH' connid = '0400' fldate = '20240115'
+                          seatsmax = 200 seatsocc = 100 ) ).
+  mo_cds_env->insert_test_data( i_data = lt_sflight ).
+
+  DATA lt_sbook TYPE TABLE OF sbook WITH EMPTY KEY.
+  lt_sbook = VALUE #(
+    ( carrid = 'LH' connid = '0400' fldate = '20240115' bookid = '0001' forcuram = 1000 )
+    ( carrid = 'LH' connid = '0400' fldate = '20240115' bookid = '0002' forcuram = 2000 )
+    ( carrid = 'LH' connid = '0400' fldate = '20240115' bookid = '0003' forcuram = 3000 ) ).
+  mo_cds_env->insert_test_data( i_data = lt_sbook ).
+
+  " Select from CDS view - aggregations will use test double data
+  SELECT * FROM zc_flight_revenue INTO TABLE @DATA(lt_result).
+
+  " Verify aggregations
+  cl_abap_unit_assert=>assert_equals(
+    exp = 3
+    act = lt_result[ 1 ]-numberofbookings
+    msg = 'Should have 3 bookings' ).
+
+  cl_abap_unit_assert=>assert_equals(
+    exp = '6000.00'
+    act = lt_result[ 1 ]-totalrevenue
+    msg = 'Total revenue should be 6000.00' ).
+ENDMETHOD.
+```
+
+### Key Classes for CDS Testing
+
+| Item | Type/Usage |
+|------|------------|
+| `CL_CDS_TEST_ENVIRONMENT` | Class with `CREATE` method |
+| `IF_CDS_TEST_ENVIRONMENT` | Interface (CREATE returns this type) |
+| `CLASS-METHODS` | `class_setup` and `class_teardown` must be static methods |
+| `CL_OSQL_TEST_ENVIRONMENT` | Test doubles for database tables (use for aggregations) |
+| `CL_ABAP_UNIT_ASSERT` | Assertions |
+
+### Key Methods
+
+| Method | Purpose |
+|--------|---------|
+| `CL_CDS_TEST_ENVIRONMENT=>create( i_for_entity = ... )` | Create test environment (returns `if_cds_test_environment`) |
+| `insert_test_data( i_data = ... )` | Insert test data into test doubles |
+| `clear_doubles` | Clear test data before each test method |
+| `destroy` | Clean up after test class |
+
+### Important Usage Notes
+
+1. **Use interface type**: `DATA mo_cds_env TYPE REF TO if_cds_test_environment` - the CREATE method returns an interface reference
+2. **CLASS-METHODS required**: `class_setup` and `class_teardown` must be declared with `CLASS-METHODS` (not `METHODS`)
+3. **Table type declaration**: Must declare `DATA lt_tab TYPE TABLE OF <type> WITH EMPTY KEY` before using `VALUE #()`
+4. **Auto-created dependencies**: CDS framework auto-creates test doubles for base tables - do not specify `i_dependency_list`
+5. **Aggregations**: For CDS views with SUM/COUNT/GROUP BY, insert test data into base tables (SFLIGHT, SCARR, etc.)
+6. **Clear doubles**: Call `clear_doubles` in `setup` method before each test
+7. **Enable associations**: Set `test_associations = 'X'` only if testing CDS associations
+8. **Exception handling**: Declare test methods with `RAISING cx_static_check` for proper exception handling
+
+### Search Reference for More Details
+
+```bash
+abapgit-agent ref "cl_cds_test_environment"
+abapgit-agent ref --topic unit-tests
+```

package/abap/guidelines/04_cds.md

@@ -0,0 +1,120 @@
+# Creating CDS Views
+
+## Creating CDS Views (DDLS)
+
+CDS views (Data Definition Language Source) require specific file naming and structure for abapGit.
+
+### File Naming
+
+CDS views require **two files**:
+
+| File | Description |
+|------|-------------|
+| `zc_my_view.ddls.asddls` | DDL source code |
+| `zc_my_view.ddls.xml` | XML metadata |
+
+**Important:** Do NOT use `.ddls.abap` extension - use `.ddls.asddls` for the source.
+
+### DDL Source File (`.ddls.asddls`)
+
+```abap
+@AbapCatalog.sqlViewName: 'ZCMYVIEW'
+@AbapCatalog.compiler.compareFilter: true
+@AccessControl.authorizationCheck: #NOT_REQUIRED
+@EndUserText.label: 'My CDS View'
+define view ZC_My_View as select from tdevc
+{
+  key devclass as Devclass,
+      parentcl as ParentPackage,
+      ctext    as Description
+}
+where devclass not like '$%'
+```
+
+### XML Metadata File (`.ddls.xml`)
+
+```xml
+<?xml version="1.0" encoding="utf-8"?>
+<abapGit version="v1.0.0" serializer="LCL_OBJECT_DDLS" serializer_version="v1.0.0">
+ <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
+  <asx:values>
+   <DDLS>
+    <DDLNAME>ZC_MY_VIEW</DDLNAME>
+    <DDLANGUAGE>E</DDLANGUAGE>
+    <DDTEXT>My CDS View</DDTEXT>
+   </DDLS>
+  </asx:values>
+ </asx:abap>
+</abapGit>
+```
+
+### Key Points
+
+1. **Avoid reserved words** - Field names like `PACKAGE`, `CLASS`, `INTERFACE` are reserved in CDS. Use alternatives like `PackageName`, `ClassName`.
+
+2. **Pull all files to activate** - When activating CDS views, use `abapgit-agent pull` (not single file) to ensure proper activation:
+   ```bash
+   abapgit-agent pull  # Pull all files
+   ```
+
+3. **System support** - CDS views require SAP systems with CDS capability (S/4HANA, SAP BW/4HANA, or ABAP 7.51+). Older systems will show error: "Object type DDLS is not supported by this system"
+
+### Debugging Activation Errors
+
+When pull shows generic errors like "Activation cancelled. Check the inactive objects":
+
+1. **Check in ADT/Eclipse** - Open the DDL source in ADT and run syntax check for detailed errors
+2. **Pull all files** - Sometimes `abapgit-agent pull` (all files) works better than single file for CDS views
+
+## Creating CDS View Entities
+
+CDS View Entities (`define view entity`) are the modern replacement for CDS Views with additional features like **associations for OData navigation**.
+
+### Differences from CDS Views
+
+| Aspect | CDS View | View Entity |
+|--------|----------|-------------|
+| Syntax | `define view` | `define view entity` |
+| Associations | No | Yes (exposed for navigation) |
+| OData/Navigation | Requires separate service | Auto-exposes associations |
+| ABAP Version | 7.40+ | 7.55+ / S/4HANA Cloud |
+
+### DDL Source File with Association
+
+```abap
+@EndUserText.label: 'Package Hierarchy'
+@AccessControl.authorizationCheck: #NOT_REQUIRED
+define view entity ZC_Pkg_Hierarchy_VE
+  as select from tdevc
+  association [0..1] to tdevc as _Parent
+    on _Parent.devclass = $projection.ParentPackage
+{
+  key devclass                           as PackageName,
+      parentcl                           as ParentPackage,
+      ctext                              as Description,
+      dlvunit                           as SoftwareComponent,
+
+      // Exposed associations
+      _Parent
+}
+where devclass not like '$%'
+```
+
+### Key Points for View Entities
+
+1. **Association syntax**: Use `$projection` to reference fields in the current entity
+2. **Association cardinality**: `[0..1]`, `[1..1]`, `[0..n]`, `[1..n]`
+3. **Expose associations**: Add the association name at the end of the SELECT to expose it for OData navigation
+4. **Activation warnings**: Search help warnings are informational and don't block activation
+
+## CDS Syntax Reference
+
+When working with CDS view syntax (arithmetic, built-in functions, aggregations, etc.):
+
+1. Run `abapgit-agent ref --topic cds` to see available topics and example files
+2. Check the example files in `abap-cheat-sheets/src/`:
+   - `zdemo_abap_cds_ve_sel.ddls.asddls` - Arithmetic expressions, built-in functions (division, cast, etc.)
+   - `zdemo_abap_cds_ve_agg_exp.ddls.asddls` - Aggregate expressions (SUM, AVG, COUNT)
+   - `zdemo_abap_cds_ve_assoc.ddls.asddls` - Associations
+
+**Note**: This requires `abap-cheat-sheets` to be in the reference folder (configured in `.abapGitAgent`).

package/abap/guidelines/05_classes.md

@@ -0,0 +1,50 @@
+# ABAP Classes and Objects
+
+## ABAP Class Definition - Must Use PUBLIC
+
+**CRITICAL**: Global ABAP classes MUST use `PUBLIC` in the class definition:
+
+```abap
+" Correct - global class
+CLASS zcl_my_class DEFINITION PUBLIC.
+  ...
+ENDCLASS.
+
+" Wrong - treated as local class, will fail activation
+CLASS zcl_my_class DEFINITION.
+  ...
+ENDCLASS.
+```
+
+**Error symptom**: `Error updating where-used list for CLAS ZCL_MY_CLASS`
+
+**Fix**: Add `PUBLIC` keyword:
+```abap
+CLASS zcl_my_class DEFINITION PUBLIC.  " <- PUBLIC required
+```
+
+## Interface Method Implementation
+
+When implementing interface methods in ABAP classes, use the interface prefix:
+
+```abap
+" Interface definition
+INTERFACE zif_my_interface PUBLIC.
+  METHODS do_something IMPORTING iv_param TYPE string.
+ENDINTERFACE.
+
+" Class implementation - use interface prefix
+CLASS zcl_my_class DEFINITION PUBLIC.
+  PUBLIC SECTION.
+    INTERFACES zif_my_interface.
+ENDCLASS.
+
+CLASS zcl_my_class IMPLEMENTATION.
+  METHOD zif_my_interface~do_something.  " <- Use interface prefix
+    " Implementation here
+  ENDMETHOD.
+ENDCLASS.
+```
+
+**Wrong**: `METHOD do_something.` - parameter `iv_param` will be unknown
+**Correct**: `METHOD zif_my_interface~do_something.` - parameters recognized