abapgit-agent 1.5.0 → 1.6.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +1 -0
- package/abap/guidelines/00_index.md +35 -0
- package/abap/guidelines/01_sql.md +72 -0
- package/abap/guidelines/02_exceptions.md +108 -0
- package/abap/guidelines/03_testing.md +252 -0
- package/abap/guidelines/04_cds.md +120 -0
- package/abap/guidelines/05_classes.md +50 -0
- package/abap/guidelines/06_objects.md +103 -0
- package/abap/guidelines/07_json.md +22 -0
- package/abap/guidelines/08_abapgit.md +193 -0
- package/bin/abapgit-agent +467 -30
- package/bin/abgagt +24 -0
- package/package.json +8 -2
- package/src/abap-client.js +65 -2
- package/src/agent.js +57 -3
- package/src/ref-search.js +989 -0
- package/.abapGitAgent.example +0 -11
- package/.github/workflows/release.yml +0 -60
- package/API.md +0 -710
- package/CLAUDE.md +0 -1058
- package/CLAUDE_MEM.md +0 -88
- package/ERROR_HANDLING.md +0 -30
- package/INSTALL.md +0 -155
- package/RELEASE_NOTES.md +0 -143
- package/abap/CLAUDE.md +0 -1010
- package/abap/copilot-instructions.md +0 -79
- package/abap/package.devc.xml +0 -10
- package/abap/zcl_abgagt_agent.clas.abap +0 -420
- package/abap/zcl_abgagt_agent.clas.xml +0 -15
- package/abap/zcl_abgagt_cmd_factory.clas.abap +0 -48
- package/abap/zcl_abgagt_cmd_factory.clas.xml +0 -15
- package/abap/zcl_abgagt_command_create.clas.abap +0 -95
- package/abap/zcl_abgagt_command_create.clas.xml +0 -15
- package/abap/zcl_abgagt_command_import.clas.abap +0 -138
- package/abap/zcl_abgagt_command_import.clas.xml +0 -15
- package/abap/zcl_abgagt_command_inspect.clas.abap +0 -456
- package/abap/zcl_abgagt_command_inspect.clas.testclasses.abap +0 -121
- package/abap/zcl_abgagt_command_inspect.clas.xml +0 -16
- package/abap/zcl_abgagt_command_preview.clas.abap +0 -386
- package/abap/zcl_abgagt_command_preview.clas.xml +0 -15
- package/abap/zcl_abgagt_command_pull.clas.abap +0 -80
- package/abap/zcl_abgagt_command_pull.clas.testclasses.abap +0 -87
- package/abap/zcl_abgagt_command_pull.clas.xml +0 -16
- package/abap/zcl_abgagt_command_tree.clas.abap +0 -237
- package/abap/zcl_abgagt_command_tree.clas.xml +0 -15
- package/abap/zcl_abgagt_command_unit.clas.abap +0 -297
- package/abap/zcl_abgagt_command_unit.clas.xml +0 -15
- package/abap/zcl_abgagt_command_view.clas.abap +0 -240
- package/abap/zcl_abgagt_command_view.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_create.clas.abap +0 -71
- package/abap/zcl_abgagt_resource_create.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_health.clas.abap +0 -25
- package/abap/zcl_abgagt_resource_health.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_import.clas.abap +0 -66
- package/abap/zcl_abgagt_resource_import.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_inspect.clas.abap +0 -63
- package/abap/zcl_abgagt_resource_inspect.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_preview.clas.abap +0 -67
- package/abap/zcl_abgagt_resource_preview.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_pull.clas.abap +0 -71
- package/abap/zcl_abgagt_resource_pull.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_tree.clas.abap +0 -70
- package/abap/zcl_abgagt_resource_tree.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_unit.clas.abap +0 -64
- package/abap/zcl_abgagt_resource_unit.clas.xml +0 -15
- package/abap/zcl_abgagt_resource_view.clas.abap +0 -68
- package/abap/zcl_abgagt_resource_view.clas.xml +0 -15
- package/abap/zcl_abgagt_rest_handler.clas.abap +0 -32
- package/abap/zcl_abgagt_rest_handler.clas.xml +0 -15
- package/abap/zcl_abgagt_util.clas.abap +0 -93
- package/abap/zcl_abgagt_util.clas.testclasses.abap +0 -84
- package/abap/zcl_abgagt_util.clas.xml +0 -16
- package/abap/zcl_abgagt_viewer_clas.clas.abap +0 -58
- package/abap/zcl_abgagt_viewer_clas.clas.xml +0 -15
- package/abap/zcl_abgagt_viewer_ddls.clas.abap +0 -83
- package/abap/zcl_abgagt_viewer_ddls.clas.xml +0 -15
- package/abap/zcl_abgagt_viewer_dtel.clas.abap +0 -98
- package/abap/zcl_abgagt_viewer_dtel.clas.xml +0 -15
- package/abap/zcl_abgagt_viewer_factory.clas.abap +0 -41
- package/abap/zcl_abgagt_viewer_factory.clas.xml +0 -15
- package/abap/zcl_abgagt_viewer_intf.clas.abap +0 -58
- package/abap/zcl_abgagt_viewer_intf.clas.xml +0 -15
- package/abap/zcl_abgagt_viewer_stru.clas.abap +0 -59
- package/abap/zcl_abgagt_viewer_stru.clas.xml +0 -15
- package/abap/zcl_abgagt_viewer_tabl.clas.abap +0 -59
- package/abap/zcl_abgagt_viewer_tabl.clas.xml +0 -15
- package/abap/zcl_abgagt_viewer_ttyp.clas.abap +0 -93
- package/abap/zcl_abgagt_viewer_ttyp.clas.xml +0 -15
- package/abap/zif_abgagt_agent.intf.abap +0 -53
- package/abap/zif_abgagt_agent.intf.xml +0 -15
- package/abap/zif_abgagt_cmd_factory.intf.abap +0 -7
- package/abap/zif_abgagt_cmd_factory.intf.xml +0 -15
- package/abap/zif_abgagt_command.intf.abap +0 -26
- package/abap/zif_abgagt_command.intf.xml +0 -15
- package/abap/zif_abgagt_util.intf.abap +0 -28
- package/abap/zif_abgagt_util.intf.xml +0 -15
- package/abap/zif_abgagt_viewer.intf.abap +0 -12
- package/abap/zif_abgagt_viewer.intf.xml +0 -15
- package/docs/commands.md +0 -142
- package/docs/create-command.md +0 -129
- package/docs/health-command.md +0 -89
- package/docs/import-command.md +0 -195
- package/docs/init-command.md +0 -189
- package/docs/inspect-command.md +0 -169
- package/docs/list-command.md +0 -289
- package/docs/preview-command.md +0 -528
- package/docs/pull-command.md +0 -202
- package/docs/status-command.md +0 -68
- package/docs/tree-command.md +0 -303
- package/docs/unit-command.md +0 -167
- package/docs/view-command.md +0 -501
- package/img/claude.png +0 -0
- package/scripts/claude-integration.js +0 -351
- package/scripts/release.js +0 -298
- package/scripts/release.sh +0 -60
- package/scripts/test-integration.js +0 -139
- package/scripts/unrelease.js +0 -277
|
@@ -1,15 +0,0 @@
|
|
|
1
|
-
<?xml version="1.0" encoding="utf-8"?>
|
|
2
|
-
<abapGit version="v1.0.0" serializer="LCL_OBJECT_CLAS" serializer_version="v1.0.0">
|
|
3
|
-
<asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
|
|
4
|
-
<asx:values>
|
|
5
|
-
<VSEOCLASS>
|
|
6
|
-
<CLSNAME>ZCL_ABGAGT_COMMAND_CREATE</CLSNAME>
|
|
7
|
-
<LANGU>E</LANGU>
|
|
8
|
-
<DESCRIPT>ABAP Git Agent - Create Command</DESCRIPT>
|
|
9
|
-
<EXPOSURE>2</EXPOSURE>
|
|
10
|
-
<STATE>1</STATE>
|
|
11
|
-
<UNICODE>X</UNICODE>
|
|
12
|
-
</VSEOCLASS>
|
|
13
|
-
</asx:values>
|
|
14
|
-
</asx:abap>
|
|
15
|
-
</abapGit>
|
|
@@ -1,138 +0,0 @@
|
|
|
1
|
-
*"*"use source
|
|
2
|
-
*"*"Local Interface:
|
|
3
|
-
*"**********************************************************************
|
|
4
|
-
CLASS zcl_abgagt_command_import DEFINITION PUBLIC FINAL CREATE PUBLIC.
|
|
5
|
-
|
|
6
|
-
PUBLIC SECTION.
|
|
7
|
-
INTERFACES zif_abgagt_command.
|
|
8
|
-
|
|
9
|
-
TYPES: BEGIN OF ty_import_params,
|
|
10
|
-
url TYPE string,
|
|
11
|
-
message TYPE string,
|
|
12
|
-
username TYPE string,
|
|
13
|
-
password TYPE string,
|
|
14
|
-
END OF ty_import_params.
|
|
15
|
-
|
|
16
|
-
ENDCLASS.
|
|
17
|
-
|
|
18
|
-
CLASS zcl_abgagt_command_import IMPLEMENTATION.
|
|
19
|
-
|
|
20
|
-
METHOD zif_abgagt_command~get_name.
|
|
21
|
-
rv_name = 'IMPORT'.
|
|
22
|
-
ENDMETHOD.
|
|
23
|
-
|
|
24
|
-
METHOD zif_abgagt_command~execute.
|
|
25
|
-
DATA: ls_params TYPE ty_import_params,
|
|
26
|
-
li_repo TYPE REF TO zif_abapgit_repo,
|
|
27
|
-
li_repo_online TYPE REF TO zif_abapgit_repo_online,
|
|
28
|
-
lo_stage TYPE REF TO zcl_abapgit_stage,
|
|
29
|
-
lt_files TYPE zif_abapgit_definitions=>ty_files_item_tt,
|
|
30
|
-
lv_package TYPE devclass,
|
|
31
|
-
lv_message TYPE string,
|
|
32
|
-
lv_files_staged TYPE i.
|
|
33
|
-
|
|
34
|
-
" Use separate variables to avoid inline DATA issues
|
|
35
|
-
DATA: li_user TYPE REF TO zif_abapgit_persist_user,
|
|
36
|
-
lv_committer_name TYPE string,
|
|
37
|
-
lv_committer_email TYPE string.
|
|
38
|
-
|
|
39
|
-
TRY.
|
|
40
|
-
IF is_param IS SUPPLIED.
|
|
41
|
-
ls_params = CORRESPONDING #( is_param ).
|
|
42
|
-
ENDIF.
|
|
43
|
-
|
|
44
|
-
" Validate URL
|
|
45
|
-
IF ls_params-url IS INITIAL.
|
|
46
|
-
rv_result = '{"success":"","error":"URL is required"}'.
|
|
47
|
-
RETURN.
|
|
48
|
-
ENDIF.
|
|
49
|
-
|
|
50
|
-
" Configure credentials if provided
|
|
51
|
-
IF ls_params-username IS NOT INITIAL AND ls_params-password IS NOT INITIAL.
|
|
52
|
-
zcl_abapgit_persist_factory=>get_user( )->set_repo_git_user_name(
|
|
53
|
-
iv_url = ls_params-url iv_username = ls_params-username ).
|
|
54
|
-
zcl_abapgit_persist_factory=>get_user( )->set_repo_login(
|
|
55
|
-
iv_url = ls_params-url iv_login = ls_params-username ).
|
|
56
|
-
zcl_abapgit_login_manager=>set_basic(
|
|
57
|
-
iv_uri = ls_params-url
|
|
58
|
-
iv_username = ls_params-username
|
|
59
|
-
iv_password = ls_params-password ).
|
|
60
|
-
ENDIF.
|
|
61
|
-
|
|
62
|
-
" Find repository by URL
|
|
63
|
-
zcl_abapgit_repo_srv=>get_instance( )->get_repo_from_url(
|
|
64
|
-
EXPORTING iv_url = ls_params-url
|
|
65
|
-
IMPORTING ei_repo = li_repo ).
|
|
66
|
-
|
|
67
|
-
IF li_repo IS NOT BOUND.
|
|
68
|
-
rv_result = '{"success":"","error":"Repository not found"}'.
|
|
69
|
-
RETURN.
|
|
70
|
-
ENDIF.
|
|
71
|
-
|
|
72
|
-
" Cast to online repo for push operations
|
|
73
|
-
li_repo_online ?= li_repo.
|
|
74
|
-
|
|
75
|
-
" Get package from repository
|
|
76
|
-
lv_package = li_repo->get_package( ).
|
|
77
|
-
|
|
78
|
-
" Build commit message if not provided
|
|
79
|
-
IF ls_params-message IS INITIAL.
|
|
80
|
-
lv_message = |feat: initial import from ABAP package { lv_package }|.
|
|
81
|
-
ELSE.
|
|
82
|
-
lv_message = ls_params-message.
|
|
83
|
-
ENDIF.
|
|
84
|
-
|
|
85
|
-
" Refresh repository
|
|
86
|
-
li_repo->refresh( ).
|
|
87
|
-
|
|
88
|
-
" Get local files
|
|
89
|
-
lt_files = li_repo->get_files_local( ).
|
|
90
|
-
|
|
91
|
-
" Count files
|
|
92
|
-
lv_files_staged = lines( lt_files ).
|
|
93
|
-
|
|
94
|
-
" Check if there are files to import
|
|
95
|
-
IF lv_files_staged = 0.
|
|
96
|
-
rv_result = '{"success":"","error":"No objects found in package"}'.
|
|
97
|
-
RETURN.
|
|
98
|
-
ENDIF.
|
|
99
|
-
|
|
100
|
-
" Create stage
|
|
101
|
-
CREATE OBJECT lo_stage.
|
|
102
|
-
|
|
103
|
-
" Add all files to stage
|
|
104
|
-
LOOP AT lt_files ASSIGNING FIELD-SYMBOL(<ls_file>).
|
|
105
|
-
lo_stage->add(
|
|
106
|
-
iv_path = <ls_file>-file-path
|
|
107
|
-
iv_filename = <ls_file>-file-filename
|
|
108
|
-
iv_data = <ls_file>-file-data ).
|
|
109
|
-
ENDLOOP.
|
|
110
|
-
|
|
111
|
-
" Get user details for committer
|
|
112
|
-
li_user = zcl_abapgit_persist_factory=>get_user( ).
|
|
113
|
-
lv_committer_name = li_user->get_default_git_user_name( ).
|
|
114
|
-
lv_committer_email = li_user->get_default_git_user_email( ).
|
|
115
|
-
|
|
116
|
-
" Prepare commit
|
|
117
|
-
DATA: ls_comment TYPE zif_abapgit_git_definitions=>ty_comment.
|
|
118
|
-
ls_comment-committer-name = lv_committer_name.
|
|
119
|
-
ls_comment-committer-email = lv_committer_email.
|
|
120
|
-
ls_comment-comment = lv_message.
|
|
121
|
-
|
|
122
|
-
" Commit and push
|
|
123
|
-
li_repo_online->push(
|
|
124
|
-
is_comment = ls_comment
|
|
125
|
-
io_stage = lo_stage ).
|
|
126
|
-
|
|
127
|
-
COMMIT WORK.
|
|
128
|
-
|
|
129
|
-
rv_result = '{"success":"X","files_staged":"' && lv_files_staged && '","commit_message":"' && lv_message && '"}'.
|
|
130
|
-
|
|
131
|
-
CATCH zcx_abapgit_exception INTO DATA(lx_error).
|
|
132
|
-
rv_result = '{"success":"","error":"' && lx_error->get_text( ) && '"}'.
|
|
133
|
-
CATCH cx_root INTO DATA(lx_other).
|
|
134
|
-
rv_result = '{"success":"","error":"' && lx_other->get_text( ) && '"}'.
|
|
135
|
-
ENDTRY.
|
|
136
|
-
ENDMETHOD.
|
|
137
|
-
|
|
138
|
-
ENDCLASS.
|
|
@@ -1,15 +0,0 @@
|
|
|
1
|
-
<?xml version="1.0" encoding="utf-8"?>
|
|
2
|
-
<abapGit version="v1.0.0" serializer="LCL_OBJECT_CLAS" serializer_version="v1.0.0">
|
|
3
|
-
<asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
|
|
4
|
-
<asx:values>
|
|
5
|
-
<VSEOCLASS>
|
|
6
|
-
<CLSNAME>ZCL_ABGAGT_COMMAND_IMPORT</CLSNAME>
|
|
7
|
-
<LANGU>E</LANGU>
|
|
8
|
-
<DESCRIPT>Import command for ABAP Git Agent</DESCRIPT>
|
|
9
|
-
<EXPOSURE>2</EXPOSURE>
|
|
10
|
-
<STATE>1</STATE>
|
|
11
|
-
<UNICODE>X</UNICODE>
|
|
12
|
-
</VSEOCLASS>
|
|
13
|
-
</asx:values>
|
|
14
|
-
</asx:abap>
|
|
15
|
-
</abapGit>
|
|
@@ -1,456 +0,0 @@
|
|
|
1
|
-
*"*"use source
|
|
2
|
-
*"*"Local Interface:
|
|
3
|
-
*"**********************************************************************
|
|
4
|
-
" INSPECT command implementation - uses SCI/SCIC for syntax check
|
|
5
|
-
CLASS zcl_abgagt_command_inspect DEFINITION PUBLIC FINAL CREATE PUBLIC.
|
|
6
|
-
PUBLIC SECTION.
|
|
7
|
-
INTERFACES zif_abgagt_command.
|
|
8
|
-
|
|
9
|
-
" Error structure for syntax check results
|
|
10
|
-
TYPES: BEGIN OF ty_error,
|
|
11
|
-
line TYPE string,
|
|
12
|
-
column TYPE string,
|
|
13
|
-
text TYPE string,
|
|
14
|
-
word TYPE string,
|
|
15
|
-
END OF ty_error.
|
|
16
|
-
|
|
17
|
-
TYPES ty_errors TYPE STANDARD TABLE OF ty_error WITH NON-UNIQUE DEFAULT KEY.
|
|
18
|
-
|
|
19
|
-
" Generic warning structure (for DDLS and other objects)
|
|
20
|
-
TYPES: BEGIN OF ty_warning,
|
|
21
|
-
type TYPE string,
|
|
22
|
-
line TYPE string,
|
|
23
|
-
column TYPE string,
|
|
24
|
-
severity TYPE string,
|
|
25
|
-
message TYPE string,
|
|
26
|
-
object_type TYPE string,
|
|
27
|
-
object_name TYPE string,
|
|
28
|
-
END OF ty_warning.
|
|
29
|
-
|
|
30
|
-
TYPES ty_warnings TYPE STANDARD TABLE OF ty_warning WITH NON-UNIQUE DEFAULT KEY.
|
|
31
|
-
|
|
32
|
-
" Result for each individual object
|
|
33
|
-
TYPES: BEGIN OF ty_inspect_result,
|
|
34
|
-
object_type TYPE string,
|
|
35
|
-
object_name TYPE string,
|
|
36
|
-
success TYPE abap_bool,
|
|
37
|
-
error_count TYPE i,
|
|
38
|
-
errors TYPE ty_errors,
|
|
39
|
-
warnings TYPE ty_warnings,
|
|
40
|
-
END OF ty_inspect_result.
|
|
41
|
-
|
|
42
|
-
" Table to hold results for multiple objects
|
|
43
|
-
TYPES ty_inspect_results TYPE STANDARD TABLE OF ty_inspect_result WITH NON-UNIQUE DEFAULT KEY.
|
|
44
|
-
|
|
45
|
-
TYPES: BEGIN OF ty_inspect_params,
|
|
46
|
-
files TYPE string_table,
|
|
47
|
-
variant TYPE string,
|
|
48
|
-
END OF ty_inspect_params.
|
|
49
|
-
|
|
50
|
-
TYPES ty_object_keys TYPE TABLE OF scir_objs WITH NON-UNIQUE DEFAULT KEY.
|
|
51
|
-
|
|
52
|
-
" Type for DDLS object names
|
|
53
|
-
TYPES ty_ddls_names TYPE STANDARD TABLE OF tadir-obj_name WITH NON-UNIQUE DEFAULT KEY.
|
|
54
|
-
|
|
55
|
-
METHODS run_syntax_check
|
|
56
|
-
IMPORTING it_objects TYPE ty_object_keys
|
|
57
|
-
iv_variant TYPE string DEFAULT 'SYNTAX_CHECK'
|
|
58
|
-
RETURNING VALUE(rt_results) TYPE ty_inspect_results.
|
|
59
|
-
|
|
60
|
-
" Validate DDLS (CDS views)
|
|
61
|
-
METHODS validate_ddls
|
|
62
|
-
IMPORTING it_ddls_names TYPE ty_ddls_names
|
|
63
|
-
RETURNING VALUE(rt_results) TYPE ty_inspect_results.
|
|
64
|
-
|
|
65
|
-
ENDCLASS.
|
|
66
|
-
|
|
67
|
-
CLASS zcl_abgagt_command_inspect IMPLEMENTATION.
|
|
68
|
-
|
|
69
|
-
METHOD zif_abgagt_command~get_name.
|
|
70
|
-
rv_name = zif_abgagt_command=>gc_inspect.
|
|
71
|
-
ENDMETHOD.
|
|
72
|
-
|
|
73
|
-
METHOD zif_abgagt_command~execute.
|
|
74
|
-
DATA: ls_params TYPE ty_inspect_params,
|
|
75
|
-
lv_file TYPE string,
|
|
76
|
-
lv_obj_type TYPE string,
|
|
77
|
-
lv_obj_name TYPE string,
|
|
78
|
-
lo_util TYPE REF TO zcl_abgagt_util,
|
|
79
|
-
lt_results TYPE ty_inspect_results,
|
|
80
|
-
lt_objects TYPE ty_object_keys,
|
|
81
|
-
lt_ddls_names TYPE ty_ddls_names,
|
|
82
|
-
ls_obj TYPE scir_objs,
|
|
83
|
-
ls_result TYPE ty_inspect_result.
|
|
84
|
-
|
|
85
|
-
" Parse parameters from is_param
|
|
86
|
-
IF is_param IS SUPPLIED.
|
|
87
|
-
ls_params = CORRESPONDING #( is_param ).
|
|
88
|
-
ENDIF.
|
|
89
|
-
|
|
90
|
-
IF ls_params-files IS INITIAL.
|
|
91
|
-
ls_result-success = abap_false.
|
|
92
|
-
ls_result-error_count = 1.
|
|
93
|
-
ls_result-object_name = 'N/A'.
|
|
94
|
-
ls_result-object_type = 'N/A'.
|
|
95
|
-
APPEND ls_result TO lt_results.
|
|
96
|
-
rv_result = /ui2/cl_json=>serialize( data = lt_results ).
|
|
97
|
-
RETURN.
|
|
98
|
-
ENDIF.
|
|
99
|
-
|
|
100
|
-
" Parse all files to objects
|
|
101
|
-
lo_util = zcl_abgagt_util=>get_instance( ).
|
|
102
|
-
|
|
103
|
-
LOOP AT ls_params-files INTO lv_file.
|
|
104
|
-
CLEAR: lv_obj_type, lv_obj_name.
|
|
105
|
-
lo_util->zif_abgagt_util~parse_file_to_object(
|
|
106
|
-
EXPORTING iv_file = lv_file
|
|
107
|
-
IMPORTING ev_obj_type = lv_obj_type
|
|
108
|
-
ev_obj_name = lv_obj_name ).
|
|
109
|
-
|
|
110
|
-
IF lv_obj_type IS NOT INITIAL AND lv_obj_name IS NOT INITIAL.
|
|
111
|
-
" Separate DDLS from other objects
|
|
112
|
-
IF lv_obj_type = 'DDLS'.
|
|
113
|
-
APPEND lv_obj_name TO lt_ddls_names.
|
|
114
|
-
ELSE.
|
|
115
|
-
CLEAR ls_obj.
|
|
116
|
-
ls_obj-objtype = lv_obj_type.
|
|
117
|
-
ls_obj-objname = lv_obj_name.
|
|
118
|
-
APPEND ls_obj TO lt_objects.
|
|
119
|
-
ENDIF.
|
|
120
|
-
ENDIF.
|
|
121
|
-
ENDLOOP.
|
|
122
|
-
|
|
123
|
-
" Run syntax check for non-DDLS objects
|
|
124
|
-
IF lt_objects IS NOT INITIAL.
|
|
125
|
-
DATA(lt_sci_results) = run_syntax_check(
|
|
126
|
-
it_objects = lt_objects
|
|
127
|
-
iv_variant = ls_params-variant ).
|
|
128
|
-
INSERT LINES OF lt_sci_results INTO TABLE lt_results.
|
|
129
|
-
ENDIF.
|
|
130
|
-
|
|
131
|
-
" Run DDLS validation
|
|
132
|
-
IF lt_ddls_names IS NOT INITIAL.
|
|
133
|
-
DATA(lt_ddls_results) = validate_ddls( lt_ddls_names ).
|
|
134
|
-
INSERT LINES OF lt_ddls_results INTO TABLE lt_results.
|
|
135
|
-
ENDIF.
|
|
136
|
-
|
|
137
|
-
" If no valid objects found
|
|
138
|
-
IF lt_objects IS INITIAL AND lt_ddls_names IS INITIAL.
|
|
139
|
-
ls_result-success = abap_false.
|
|
140
|
-
ls_result-error_count = 1.
|
|
141
|
-
ls_result-object_name = 'N/A'.
|
|
142
|
-
ls_result-object_type = 'N/A'.
|
|
143
|
-
APPEND ls_result TO lt_results.
|
|
144
|
-
ENDIF.
|
|
145
|
-
|
|
146
|
-
rv_result = /ui2/cl_json=>serialize( data = lt_results ).
|
|
147
|
-
ENDMETHOD.
|
|
148
|
-
|
|
149
|
-
METHOD validate_ddls.
|
|
150
|
-
" Validate DDLS (CDS views) using CL_DD_DDL_HANDLER_FACTORY
|
|
151
|
-
" First checks inactive version ('M'), falls back to active ('A')
|
|
152
|
-
DATA: lv_ddls_name TYPE ddlname,
|
|
153
|
-
lo_handler TYPE REF TO if_dd_ddl_handler,
|
|
154
|
-
ls_ddlsrcv TYPE ddddlsrcv,
|
|
155
|
-
ls_error TYPE ty_error,
|
|
156
|
-
ls_warning TYPE ty_warning,
|
|
157
|
-
lt_warnings TYPE ddl2ddicwarnings,
|
|
158
|
-
lx_error TYPE REF TO cx_dd_ddl_check,
|
|
159
|
-
lv_found TYPE abap_bool,
|
|
160
|
-
ls_result TYPE ty_inspect_result,
|
|
161
|
-
lv_err_msg TYPE string,
|
|
162
|
-
lv_warn_msg TYPE string.
|
|
163
|
-
|
|
164
|
-
rt_results = VALUE #( ).
|
|
165
|
-
|
|
166
|
-
" Create DDL handler
|
|
167
|
-
lo_handler = cl_dd_ddl_handler_factory=>create( ).
|
|
168
|
-
|
|
169
|
-
" Check each DDLS object
|
|
170
|
-
LOOP AT it_ddls_names INTO lv_ddls_name.
|
|
171
|
-
CLEAR: ls_ddlsrcv, lt_warnings, lv_found, ls_result.
|
|
172
|
-
|
|
173
|
-
ls_result-object_type = 'DDLS'.
|
|
174
|
-
ls_result-object_name = lv_ddls_name.
|
|
175
|
-
ls_result-success = abap_true.
|
|
176
|
-
|
|
177
|
-
" First try to read inactive version (get_state = 'M')
|
|
178
|
-
TRY.
|
|
179
|
-
lo_handler->read(
|
|
180
|
-
EXPORTING
|
|
181
|
-
name = lv_ddls_name
|
|
182
|
-
get_state = 'M'
|
|
183
|
-
IMPORTING
|
|
184
|
-
ddddlsrcv_wa = ls_ddlsrcv ).
|
|
185
|
-
|
|
186
|
-
IF ls_ddlsrcv-source IS NOT INITIAL.
|
|
187
|
-
lv_found = abap_true.
|
|
188
|
-
ENDIF.
|
|
189
|
-
|
|
190
|
-
CATCH cx_dd_ddl_check.
|
|
191
|
-
" Ignore - will try active version
|
|
192
|
-
ENDTRY.
|
|
193
|
-
|
|
194
|
-
" If no inactive version, try active version
|
|
195
|
-
IF lv_found = abap_false.
|
|
196
|
-
TRY.
|
|
197
|
-
lo_handler->read(
|
|
198
|
-
EXPORTING
|
|
199
|
-
name = lv_ddls_name
|
|
200
|
-
get_state = 'A'
|
|
201
|
-
IMPORTING
|
|
202
|
-
ddddlsrcv_wa = ls_ddlsrcv ).
|
|
203
|
-
|
|
204
|
-
IF ls_ddlsrcv-source IS NOT INITIAL.
|
|
205
|
-
lv_found = abap_true.
|
|
206
|
-
ENDIF.
|
|
207
|
-
CATCH cx_dd_ddl_check.
|
|
208
|
-
" Ignore
|
|
209
|
-
ENDTRY.
|
|
210
|
-
ENDIF.
|
|
211
|
-
|
|
212
|
-
" If still not found, report error
|
|
213
|
-
IF lv_found = abap_false.
|
|
214
|
-
ls_error-line = '1'.
|
|
215
|
-
ls_error-column = '1'.
|
|
216
|
-
ls_error-text = |DDLS { lv_ddls_name } not found in repository|.
|
|
217
|
-
APPEND ls_error TO ls_result-errors.
|
|
218
|
-
ls_result-success = abap_false.
|
|
219
|
-
ls_result-error_count = lines( ls_result-errors ).
|
|
220
|
-
APPEND ls_result TO rt_results.
|
|
221
|
-
CONTINUE.
|
|
222
|
-
ENDIF.
|
|
223
|
-
|
|
224
|
-
" Run validation check
|
|
225
|
-
TRY.
|
|
226
|
-
" Get warnings from check method
|
|
227
|
-
lo_handler->check(
|
|
228
|
-
EXPORTING
|
|
229
|
-
name = lv_ddls_name
|
|
230
|
-
IMPORTING
|
|
231
|
-
warnings = lt_warnings
|
|
232
|
-
CHANGING
|
|
233
|
-
ddlsrcv_wa = ls_ddlsrcv ).
|
|
234
|
-
|
|
235
|
-
" Parse warnings from check method
|
|
236
|
-
LOOP AT lt_warnings INTO DATA(ls_warn_from_check).
|
|
237
|
-
CLEAR ls_warning.
|
|
238
|
-
ls_warning-type = ls_warn_from_check-type.
|
|
239
|
-
ls_warning-line = ls_warn_from_check-line.
|
|
240
|
-
ls_warning-column = ls_warn_from_check-column.
|
|
241
|
-
ls_warning-severity = ls_warn_from_check-severity.
|
|
242
|
-
ls_warning-object_type = 'DDLS'.
|
|
243
|
-
ls_warning-object_name = lv_ddls_name.
|
|
244
|
-
" Use MESSAGE statement to get real warning message
|
|
245
|
-
MESSAGE ID ls_warn_from_check-arbgb TYPE 'E' NUMBER ls_warn_from_check-msgnr
|
|
246
|
-
WITH ls_warn_from_check-var1 ls_warn_from_check-var2 ls_warn_from_check-var3 ls_warn_from_check-var4
|
|
247
|
-
INTO lv_warn_msg.
|
|
248
|
-
ls_warning-message = lv_warn_msg.
|
|
249
|
-
APPEND ls_warning TO ls_result-warnings.
|
|
250
|
-
ENDLOOP.
|
|
251
|
-
|
|
252
|
-
" If no errors and no warnings, validation passed
|
|
253
|
-
IF ls_result-warnings IS INITIAL.
|
|
254
|
-
ls_result-success = abap_true.
|
|
255
|
-
ELSE.
|
|
256
|
-
ls_result-success = abap_false.
|
|
257
|
-
ENDIF.
|
|
258
|
-
|
|
259
|
-
CATCH cx_dd_ddl_check INTO lx_error.
|
|
260
|
-
" Validation failed - get error details using get_errors method
|
|
261
|
-
DATA(lt_errors) = lx_error->get_errors( ).
|
|
262
|
-
LOOP AT lt_errors INTO DATA(ls_err).
|
|
263
|
-
CLEAR ls_error.
|
|
264
|
-
ls_error-line = ls_err-line.
|
|
265
|
-
ls_error-column = ls_err-column.
|
|
266
|
-
" Use MESSAGE statement to get real error message
|
|
267
|
-
MESSAGE ID ls_err-arbgb TYPE 'E' NUMBER ls_err-msgnr
|
|
268
|
-
WITH ls_err-var1 ls_err-var2 ls_err-var3 ls_err-var4
|
|
269
|
-
INTO lv_err_msg.
|
|
270
|
-
ls_error-text = lv_err_msg.
|
|
271
|
-
APPEND ls_error TO ls_result-errors.
|
|
272
|
-
ENDLOOP.
|
|
273
|
-
|
|
274
|
-
" Also get warnings if any
|
|
275
|
-
DATA(lt_warn) = lx_error->get_warnings( ).
|
|
276
|
-
LOOP AT lt_warn INTO DATA(ls_warn2).
|
|
277
|
-
CLEAR ls_warning.
|
|
278
|
-
ls_warning-type = ls_warn2-type.
|
|
279
|
-
ls_warning-line = ls_warn2-line.
|
|
280
|
-
ls_warning-column = ls_warn2-column.
|
|
281
|
-
ls_warning-severity = ls_warn2-severity.
|
|
282
|
-
ls_warning-object_type = 'DDLS'.
|
|
283
|
-
ls_warning-object_name = lv_ddls_name.
|
|
284
|
-
" Use MESSAGE statement to get real warning message
|
|
285
|
-
MESSAGE ID ls_warn2-arbgb TYPE 'E' NUMBER ls_warn2-msgnr
|
|
286
|
-
WITH ls_warn2-var1 ls_warn2-var2 ls_warn2-var3 ls_warn2-var4
|
|
287
|
-
INTO lv_warn_msg.
|
|
288
|
-
ls_warning-message = lv_warn_msg.
|
|
289
|
-
APPEND ls_warning TO ls_result-warnings.
|
|
290
|
-
ENDLOOP.
|
|
291
|
-
|
|
292
|
-
ls_result-success = abap_false.
|
|
293
|
-
ENDTRY.
|
|
294
|
-
|
|
295
|
-
ls_result-error_count = lines( ls_result-errors ).
|
|
296
|
-
APPEND ls_result TO rt_results.
|
|
297
|
-
ENDLOOP.
|
|
298
|
-
ENDMETHOD.
|
|
299
|
-
|
|
300
|
-
METHOD run_syntax_check.
|
|
301
|
-
DATA: lv_name TYPE sci_objs,
|
|
302
|
-
lo_objset TYPE REF TO cl_ci_objectset,
|
|
303
|
-
lo_variant TYPE REF TO cl_ci_checkvariant,
|
|
304
|
-
lo_inspection TYPE REF TO cl_ci_inspection,
|
|
305
|
-
lt_list TYPE scit_alvlist,
|
|
306
|
-
ls_error TYPE ty_error,
|
|
307
|
-
ls_warning TYPE ty_warning,
|
|
308
|
-
ls_list TYPE scir_alvlist,
|
|
309
|
-
lx_error TYPE REF TO cx_root,
|
|
310
|
-
ls_result TYPE ty_inspect_result.
|
|
311
|
-
|
|
312
|
-
rt_results = VALUE #( ).
|
|
313
|
-
|
|
314
|
-
TRY.
|
|
315
|
-
" Create unique name for inspection
|
|
316
|
-
CONCATENATE 'SYNT_' sy-uname sy-datum sy-uzeit INTO lv_name.
|
|
317
|
-
|
|
318
|
-
" Create object set
|
|
319
|
-
lo_objset = cl_ci_objectset=>save_from_list(
|
|
320
|
-
p_name = lv_name
|
|
321
|
-
p_objects = it_objects ).
|
|
322
|
-
|
|
323
|
-
" Get check variant (default: SYNTAX_CHECK)
|
|
324
|
-
DATA lv_variant TYPE sci_chkv.
|
|
325
|
-
lv_variant = iv_variant.
|
|
326
|
-
IF lv_variant IS INITIAL.
|
|
327
|
-
lv_variant = 'SYNTAX_CHECK'.
|
|
328
|
-
ENDIF.
|
|
329
|
-
|
|
330
|
-
" Try to get the variant using EXCEPTIONS
|
|
331
|
-
cl_ci_checkvariant=>get_ref(
|
|
332
|
-
EXPORTING
|
|
333
|
-
p_user = ''
|
|
334
|
-
p_name = lv_variant
|
|
335
|
-
RECEIVING
|
|
336
|
-
p_ref = lo_variant
|
|
337
|
-
EXCEPTIONS
|
|
338
|
-
chkv_not_exists = 1
|
|
339
|
-
missing_parameter = 2
|
|
340
|
-
broken_variant = 3
|
|
341
|
-
OTHERS = 4 ).
|
|
342
|
-
|
|
343
|
-
IF sy-subrc <> 0.
|
|
344
|
-
" Variant not found, return error
|
|
345
|
-
ls_result-object_type = 'VARIANT'.
|
|
346
|
-
ls_result-object_name = lv_variant.
|
|
347
|
-
ls_result-success = abap_false.
|
|
348
|
-
ls_result-error_count = 1.
|
|
349
|
-
APPEND VALUE #( line = '0' column = '0' text = |Check variant "{ lv_variant }" not found (rc={ sy-subrc })| word = '' ) TO ls_result-errors.
|
|
350
|
-
APPEND ls_result TO rt_results.
|
|
351
|
-
RETURN.
|
|
352
|
-
ENDIF.
|
|
353
|
-
|
|
354
|
-
" Create inspection
|
|
355
|
-
cl_ci_inspection=>create(
|
|
356
|
-
EXPORTING
|
|
357
|
-
p_user = sy-uname
|
|
358
|
-
p_name = lv_name
|
|
359
|
-
RECEIVING
|
|
360
|
-
p_ref = lo_inspection ).
|
|
361
|
-
|
|
362
|
-
" Set inspection with object set and variant
|
|
363
|
-
lo_inspection->set(
|
|
364
|
-
EXPORTING
|
|
365
|
-
p_chkv = lo_variant
|
|
366
|
-
p_objs = lo_objset ).
|
|
367
|
-
|
|
368
|
-
" Save inspection
|
|
369
|
-
lo_inspection->save( ).
|
|
370
|
-
|
|
371
|
-
" Run inspection
|
|
372
|
-
lo_inspection->run(
|
|
373
|
-
EXPORTING
|
|
374
|
-
p_howtorun = 'R'
|
|
375
|
-
EXCEPTIONS
|
|
376
|
-
invalid_check_version = 1
|
|
377
|
-
OTHERS = 2 ).
|
|
378
|
-
|
|
379
|
-
" Get results
|
|
380
|
-
lo_inspection->plain_list( IMPORTING p_list = lt_list ).
|
|
381
|
-
|
|
382
|
-
" Build result for each object
|
|
383
|
-
LOOP AT it_objects INTO DATA(ls_obj).
|
|
384
|
-
CLEAR ls_result.
|
|
385
|
-
ls_result-object_type = ls_obj-objtype.
|
|
386
|
-
ls_result-object_name = ls_obj-objname.
|
|
387
|
-
ls_result-success = abap_true.
|
|
388
|
-
|
|
389
|
-
" Get errors and warnings for this object
|
|
390
|
-
LOOP AT lt_list INTO ls_list WHERE objname = ls_obj-objname.
|
|
391
|
-
" Check severity - 'E' = Error, 'W' = Warning, 'I' = Info
|
|
392
|
-
IF ls_list-kind = 'E'.
|
|
393
|
-
" Error
|
|
394
|
-
CLEAR ls_error.
|
|
395
|
-
ls_error-line = ls_list-line.
|
|
396
|
-
ls_error-column = ls_list-col.
|
|
397
|
-
ls_error-text = ls_list-text.
|
|
398
|
-
ls_error-word = ls_list-code.
|
|
399
|
-
APPEND ls_error TO ls_result-errors.
|
|
400
|
-
ELSEIF ls_list-kind = 'W' OR ls_list-kind = 'I'.
|
|
401
|
-
" Warning or Info
|
|
402
|
-
CLEAR ls_warning.
|
|
403
|
-
ls_warning-line = ls_list-line.
|
|
404
|
-
ls_warning-column = ls_list-col.
|
|
405
|
-
ls_warning-severity = ls_list-kind.
|
|
406
|
-
ls_warning-message = ls_list-text.
|
|
407
|
-
ls_warning-object_type = ls_obj-objtype.
|
|
408
|
-
ls_warning-object_name = ls_obj-objname.
|
|
409
|
-
APPEND ls_warning TO ls_result-warnings.
|
|
410
|
-
ENDIF.
|
|
411
|
-
ENDLOOP.
|
|
412
|
-
|
|
413
|
-
ls_result-error_count = lines( ls_result-errors ).
|
|
414
|
-
IF ls_result-error_count > 0.
|
|
415
|
-
ls_result-success = abap_false.
|
|
416
|
-
ENDIF.
|
|
417
|
-
|
|
418
|
-
APPEND ls_result TO rt_results.
|
|
419
|
-
ENDLOOP.
|
|
420
|
-
|
|
421
|
-
" Cleanup
|
|
422
|
-
lo_inspection->delete(
|
|
423
|
-
EXCEPTIONS
|
|
424
|
-
locked = 1
|
|
425
|
-
error_in_enqueue = 2
|
|
426
|
-
not_authorized = 3
|
|
427
|
-
exceptn_appl_exists = 4
|
|
428
|
-
OTHERS = 5 ).
|
|
429
|
-
|
|
430
|
-
lo_objset->delete(
|
|
431
|
-
EXCEPTIONS
|
|
432
|
-
exists_in_insp = 1
|
|
433
|
-
locked = 2
|
|
434
|
-
error_in_enqueue = 3
|
|
435
|
-
not_authorized = 4
|
|
436
|
-
exists_in_objs = 5
|
|
437
|
-
OTHERS = 6 ).
|
|
438
|
-
|
|
439
|
-
CATCH cx_root INTO lx_error.
|
|
440
|
-
" Return error for all objects
|
|
441
|
-
LOOP AT it_objects INTO ls_obj.
|
|
442
|
-
CLEAR ls_result.
|
|
443
|
-
ls_result-object_type = ls_obj-objtype.
|
|
444
|
-
ls_result-object_name = ls_obj-objname.
|
|
445
|
-
ls_result-success = abap_false.
|
|
446
|
-
ls_error-line = '1'.
|
|
447
|
-
ls_error-column = '1'.
|
|
448
|
-
ls_error-text = lx_error->get_text( ).
|
|
449
|
-
APPEND ls_error TO ls_result-errors.
|
|
450
|
-
ls_result-error_count = 1.
|
|
451
|
-
APPEND ls_result TO rt_results.
|
|
452
|
-
ENDLOOP.
|
|
453
|
-
ENDTRY.
|
|
454
|
-
ENDMETHOD.
|
|
455
|
-
|
|
456
|
-
ENDCLASS.
|