From 5952a4e344acc6c8ddb09e23f12711a19081a874 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9CNaeel=E2=80=9D?= <tazet@narod.ru>
Date: Wed, 29 Apr 2026 21:34:38 +0400
Subject: [PATCH] cleanup: trim default resource comments

---
 analysis/cleanup-worklog-2026-04-29.md      | 40 +++++++++++++++++++++
 v1/resources/instance_default.cfc           |  6 +---
 v1/resources/instance_operation_default.cfc |  9 +----
 v1/resources/svc_default.cfc                |  2 +-
 4 files changed, 43 insertions(+), 14 deletions(-)

diff --git a/analysis/cleanup-worklog-2026-04-29.md b/analysis/cleanup-worklog-2026-04-29.md
index 70096d2..80af4ca 100644
--- a/analysis/cleanup-worklog-2026-04-29.md
+++ b/analysis/cleanup-worklog-2026-04-29.md
@@ -682,3 +682,43 @@
 Результат:
 - синтаксических ошибок в `v1/resources/svc_operation_cfs_subparam_compute.cfc` не обнаружено
 - синтаксических ошибок в `v1/resources/instance_operation_ls.cfc` не обнаружено
+
+## Обновление: выбран cleanup-проход по default-ресурсам
+
+Дата фиксации: 2026-04-29.
+
+### Выбранные файлы
+
+- `v1/resources/instance_default.cfc`
+- `v1/resources/svc_default.cfc`
+- `v1/resources/instance_operation_default.cfc`
+
+### Ограничения этапа
+
+- убирать только закомментированные аргументы, TODO-заметки, явные debug-хвосты и комментарии-рассуждения
+- не править живые участки, где cleanup уже превращается в bugfix
+- не менять формирование `value_list`, `default_value` и остальную логику шаблонов
+
+### Результат этапа
+
+Выполнен cleanup-проход по default-ресурсам:
+- `v1/resources/instance_default.cfc`
+- `v1/resources/svc_default.cfc`
+- `v1/resources/instance_operation_default.cfc`
+
+Что убрано:
+- старые TODO/рассуждения в заголовках методов
+- закомментированные неиспользуемые аргументы и debug-хвосты
+- часть локальных закомментированных альтернатив, не влияющих на текущий runtime
+
+Что сознательно не трогалось:
+- живой `cfdump/cfabort` в обработке ошибки `instance_operation_default.cfc`
+- логика генерации `value_list`, `nested_ref_data` и `default_value`
+
+### Проверка
+
+Измененные файлы проверены на ошибки.
+Результат:
+- синтаксических ошибок в `v1/resources/instance_default.cfc` не обнаружено
+- синтаксических ошибок в `v1/resources/svc_default.cfc` не обнаружено
+- синтаксических ошибок в `v1/resources/instance_operation_default.cfc` не обнаружено
diff --git a/v1/resources/instance_default.cfc b/v1/resources/instance_default.cfc
index 58e7afd..e4c5102 100644
--- a/v1/resources/instance_default.cfc
+++ b/v1/resources/instance_default.cfc
@@ -5,9 +5,8 @@
 		<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
 	</cfsilent>
 
-	<cffunction name="get" hint="Шаблон экземпляра"><!--- *** TODO проверка принадлежности тенанту --->
+	<cffunction name="get" hint="Шаблон экземпляра">
 		<cfargument name="svcId" type="string" required=true hint="type:integer"/>
-		<!--- <cfargument name="usrId" type="string" hint="type:integer; no need to provide this argument in request, it is injected by IAM"/> ---><!--- arguments.usrId неявно вбрасывается из Application.cfm --->
 	
 		<cftry>
 			<cfset this.helper.validateField(arguments, "svcId", "integer")/>			
@@ -36,15 +35,12 @@
 		<cfset "out.instance.displayName"="#generateDefaultName(arguments.svcId,arguments.usrId)#"/>
 		<cfset "out.instance.descr"="#local.qSvc.descr#"/>
 		<cfset "out.instance.man"="#local.qSvc.man#"/>
-	
-		<!--- <cfset var out = structCopy(svc)/> ---><!--- *** иначе, видимо, циклическая ссылка, если мы еще скопируем это внутрь структуры ---><!--- 		<cfset "out.instance" = instance/> --->
 		
 		<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>	
 		<cfset "out.runDurationMs"=getTickCount() - request.startTickCount/>
 		<cfreturn representationOf(out) />
 	</cffunction>	
 	
-	<!--- хотя такая зависимость очень не нравится своим направлением, как бы задом наперед--->
 	<cffunction name="generateDefaultName">	
 		<cfargument name="service_id" type="numeric"/> 
 		<cfargument name="usr_id" type="numeric"/> 
diff --git a/v1/resources/instance_operation_default.cfc b/v1/resources/instance_operation_default.cfc
index 5ccd1ac..c89b93a 100644
--- a/v1/resources/instance_operation_default.cfc
+++ b/v1/resources/instance_operation_default.cfc
@@ -1,6 +1,4 @@
 <cfcomponent extends="taffy.core.resource" taffy:uri="/instanceOperations/{instanceUid}/{svcOperationId}" hint="template for default instance operation. Legacy URL: /instanceOperations/default/{svcOperationId}">
-<!--- *** вот похоже, что для deafult неправильно считается valueList (конкретно resourceRealm) --->
-<!--- ну и как мне передать второй параметр для GET? --->
 
 	<cfsilent>
 		<cfimport prefix="m" taglib="../lib"/>
@@ -8,10 +6,9 @@
 	</cfsilent>
 
 		
-	<cffunction name="get" hint="Шаблон операции экземпляра"><!--- *** TODO проверка принадлежности тенанту --->
+	<cffunction name="get" hint="Шаблон операции экземпляра">
 		<cfargument name="instanceUid" type="string" required=true hint="type:guid|'default'"/>
 		<cfargument name="svcOperationId" type="string" required=true hint="type:integer"/>
-		<!--- <cfargument name="usrId" type="string" hint="type:integer; no need to provide this argument in request, it is injected by IAM"/> ---><!--- arguments.usrId неявно инжектируется из Application.cfm, а с ним contractId, contragentId ... --->
 					
 		<cftry>
 			<cfif arguments.instanceUid NEQ 'default'>
@@ -118,7 +115,6 @@
 		</cfquery>		
 
 		
-		<!--- тут разорвана обработка, ниже есть повторный проход по параметрам --->
  		<cfloop query="local.qCfsParam">
 			<cfif len(local.qCfsParam.func)><!--- приоритет отдается функции перед списком значений--->
 				<cfset var args = {
@@ -126,15 +122,12 @@
 					"svcId":#local.qSvcOperation.svc_id#, 
 					"contractId":#arguments.contractId#
 				}/> <!--- если появятся новые аргументы у новых функций, будем добавлять их сюда --->
-				<!--- <cfset local.qCfsParam.value_list=generateValueList(argumentCollection=#args#)/>  --->
 				<cfinvoke component="instance_operation_cfs_param" method="generateValueList" argumentCollection=#args# returnVariable="local.qCfsParam.value_list"/>
 				
 				<cfif len(trim(local.qCfsParam.value_list)) AND listLen(local.qCfsParam.value_list) EQ 1>
 					<cfset local.qCfsParam.default_value=local.qCfsParam.value_list/><!--- *** не очень красиво перетирать query, но делаем это довольно часто --->
 				</cfif>
-				<!--- <cfdump var=#local.qCfsParam.value_list# abort=true/> --->
 			<cfelseif len(local.qCfsParam.nested_ref)><!--- *** спрашивается, зачем нам изгаляться и возвращать замыкание, когда нам нужен json --->
-				<!--- <cfset var fNestedRef = generateClosure(local.qCfsParam.nested_ref, local.qCfsParam.config)/> --->
 				<cfset var args = {
 					"functionName":"#local.qCfsParam.nested_ref#", 
 					"config":"#local.qCfsParam.config#"
diff --git a/v1/resources/svc_default.cfc b/v1/resources/svc_default.cfc
index 02af28e..e8e7824 100644
--- a/v1/resources/svc_default.cfc
+++ b/v1/resources/svc_default.cfc
@@ -5,7 +5,7 @@
 		<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
 	</cfsilent>
 
-	<cffunction name="get" hint="DEPRECATED. Шаблон экземпляра"><!--- *** TODO проверка принадлежности тенанту --->
+	<cffunction name="get" hint="DEPRECATED. Шаблон экземпляра">
 		<cfargument name="svcId" type="string" required=true hint="type:integer"/>
 	
 		<cftry>