removed extra comments

Author: Евгений Балякин

cmd/pgkernel/main.go

@@ -14,22 +14,6 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// FILE:cmd/pgkernel/main.go
-// VERSION:1.0.1
-// START_MODULE_CONTRACT:
-// PURPOSE:Provide CLI entrypoint for pgkernel check/version commands.
-// SCOPE:Flag parsing, detection orchestration, policy application, output rendering, process exit.
-// INPUT:CLI flags and runtime host state.
-// OUTPUT:Terminal/JSON/Markdown reports and deterministic exit codes.
-// KEYWORDS:[DOMAIN(CLI): cobra commands; CONCEPT(Policy): CI thresholds; TECH(Go): single binary]
-// LINKS:[USES_API(internal/*): detect/check/policy/output pipeline]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.1 - Added runtime-error aware exit code path and merged baseline/compare regression integration.
-// PREV_CHANGE_SUMMARY:1.0.0 - Implemented full CLI orchestration for pgkernel v0.1.
-// END_CHANGE_SUMMARY
-
 type checkOptions struct {
 	format      string
 	pgConfig    string
@@ -131,7 +115,6 @@ func runCheck(opts checkOptions) int {
 	}
 	report.Regressions = dedupeRegressions(report.Regressions)
 
-	// BUG_FIX_CONTEXT: Runtime checker panics were previously converted into normal check outcomes, which could hide hard failures. The runtimeError signal now propagates into exit code priority logic.
 	exitCode := policy.DetermineExitCode(results, report.Regressions, opts.failOn, runtimeError, regressionsOnly)
 	report.Summary = checker.BuildSummary(results)
 	report.Summary.ExitCode = exitCode

internal/checker/checker.go

@@ -5,22 +5,6 @@ import (
 	"sort"
 )
 
-// FILE:internal/checker/checker.go
-// VERSION:1.0.1
-// START_MODULE_CONTRACT:
-// PURPOSE:Provide check interface and deterministic runner execution order.
-// SCOPE:Check metadata contract, runner assembly, status-to-severity mapping.
-// INPUT:RuntimeState snapshot and check implementations.
-// OUTPUT:Ordered list of CheckResult entries.
-// KEYWORDS:[PATTERN(Strategy): pluggable checks; PATTERN(Pipeline): evaluation runner]
-// LINKS:[READS_DATA_FROM(internal/checker/context.go): RuntimeState]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.1 - Added panic recovery, runtime failure propagation, and result normalization to preserve contract stability.
-// PREV_CHANGE_SUMMARY:1.0.0 - Implemented check strategy interface and deterministic execution.
-// END_CHANGE_SUMMARY
-
 type Meta struct {
 	ID       string
 	Name     string
@@ -37,18 +21,6 @@ type Runner struct {
 	runtimeError bool
 }
 
-// START_FUNCTION_NewRunner
-// START_CONTRACT:
-// PURPOSE:Create a runner that executes checks in deterministic ID order.
-// INPUTS:
-// - provided checks => checks: []Check
-// OUTPUTS:
-// - *Runner - ready-to-run checker engine
-// SIDE_EFFECTS: none
-// KEYWORDS:[PATTERN(Sorting): stable execution; CONCEPT(Reproducibility): deterministic output]
-// LINKS:[USES_API(sort.Slice): ordering by check id]
-// COMPLEXITY_SCORE:[3][single sort operation]
-// END_CONTRACT
 func NewRunner(checks []Check) *Runner {
 	ordered := make([]Check, len(checks))
 	copy(ordered, checks)
@@ -58,18 +30,6 @@ func NewRunner(checks []Check) *Runner {
 	return &Runner{checks: ordered}
 }
 
-// START_FUNCTION_Run
-// START_CONTRACT:
-// PURPOSE:Execute all registered checks against runtime state.
-// INPUTS:
-// - runtime snapshot => state: RuntimeState
-// OUTPUTS:
-// - []CheckResult - sorted check result rows
-// SIDE_EFFECTS: none
-// KEYWORDS:[PATTERN(Pipeline): sequential execution; CONCEPT(Stateless): pure evaluation]
-// LINKS:[USES_API(None): none]
-// COMPLEXITY_SCORE:[3][single loop over checks]
-// END_CONTRACT
 func (r *Runner) Run(state RuntimeState) []CheckResult {
 	r.runtimeError = false
 	results := make([]CheckResult, 0, len(r.checks))
@@ -86,18 +46,6 @@ func (r *Runner) Run(state RuntimeState) []CheckResult {
 	return results
 }
 
-// START_FUNCTION_HasRuntimeError
-// START_CONTRACT:
-// PURPOSE:Expose whether any checker execution produced runtime failure.
-// INPUTS:
-// - none
-// OUTPUTS:
-// - bool - true when at least one checker panic was recovered
-// SIDE_EFFECTS: none
-// KEYWORDS:[CONCEPT(Reliability): runtime signal propagation]
-// LINKS:[USES_API(None): none]
-// COMPLEXITY_SCORE:[1][field accessor]
-// END_CONTRACT
 func (r *Runner) HasRuntimeError() bool {
 	return r.runtimeError
 }
@@ -143,7 +91,6 @@ func runtimeFailureResult(meta Meta, details string) CheckResult {
 }
 
 func normalizeCheckResult(meta Meta, result CheckResult) CheckResult {
-	// BUG_FIX_CONTEXT: Earlier checker outputs could miss required contract fields, which broke JSON stability and safety classification. The normalizer guarantees complete machine-readable rows for every check.
 	if result.ID == "" {
 		result.ID = meta.ID
 	}
@@ -183,18 +130,6 @@ func normalizeCheckResult(meta Meta, result CheckResult) CheckResult {
 	return result
 }
 
-// START_FUNCTION_SeverityFromStatus
-// START_CONTRACT:
-// PURPOSE:Normalize output severity from resulting status.
-// INPUTS:
-// - status emitted by check => status: Status
-// OUTPUTS:
-// - Severity - mapped urgency level
-// SIDE_EFFECTS: none
-// KEYWORDS:[PATTERN(Mapping): status normalization]
-// LINKS:[USES_API(None): none]
-// COMPLEXITY_SCORE:[2][constant switch]
-// END_CONTRACT
 func SeverityFromStatus(status Status) Severity {
 	switch status {
 	case StatusCrit:

internal/checker/context.go

@@ -1,21 +1,5 @@
 package checker
 
-// FILE:internal/checker/context.go
-// VERSION:1.0.0
-// START_MODULE_CONTRACT:
-// PURPOSE:Define normalized runtime state consumed by all check modules.
-// SCOPE:Kernel, system, and PostgreSQL signals collected by detect layer.
-// INPUT:System and configuration probes from detect package.
-// OUTPUT:Stable in-memory state for deterministic check evaluation and tests.
-// KEYWORDS:[DOMAIN(Diagnostics): runtime snapshot; CONCEPT(Testability): dependency isolation]
-// LINKS:[READS_DATA_FROM(detect): detector outputs]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.0 - Added strongly typed runtime state model for all checks.
-// PREV_CHANGE_SUMMARY:none
-// END_CHANGE_SUMMARY
-
 type KernelState struct {
 	Version           string
 	Major             int

internal/checker/result.go

@@ -6,28 +6,6 @@ import (
 	"github.com/google/uuid"
 )
 
-// FILE:internal/checker/result.go
-// VERSION:1.0.0
-// START_MODULE_CONTRACT:
-// PURPOSE:Define stable report contracts shared across runner, policy, and output layers.
-// SCOPE:Check result schema, report envelope, summary counters, compatibility declarations.
-// INPUT:Check evaluation artifacts from checker modules.
-// OUTPUT:Machine-readable report model for pretty/json/markdown renderers.
-// KEYWORDS:[DOMAIN(Observability): report contract; CONCEPT(Compatibility): schema stability; TECH(Go): typed enums]
-// LINKS:[USES_API(JSON): encoding/json; READS_DATA_FROM(Runtime): checker context]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.0 - Introduced canonical result and report models for pgkernel v0.1 implementation.
-// PREV_CHANGE_SUMMARY:none
-// END_CHANGE_SUMMARY
-
-// START_MODULE_MAP:
-// TYPE [Weight 1-10][Entity description in English] =>[entity_name_latin]
-// FUNC [9][Build report envelope with deterministic metadata] => [NewReport]
-// FUNC [8][Initialize summary counters for report checks] => [BuildSummary]
-// END_MODULE_MAP
-
 const (
 	SchemaVersion = "1.0.0"
 	ToolVersion   = "0.1.0"
@@ -149,21 +127,6 @@ type Report struct {
 	Regressions   []Regression   `json:"regressions,omitempty"`
 }
 
-// START_FUNCTION_NewReport
-// START_CONTRACT:
-// PURPOSE:Construct a complete report envelope with deterministic schema metadata.
-// INPUTS:
-// - profile runtime profile string => profile: string
-// - detected system facts => system: SystemInfo
-// - detected postgres facts => pg: PostgreSQLInfo
-// - evaluated check rows => checks: []CheckResult
-// OUTPUTS:
-// - Report - report with generated id and RFC3339 timestamp
-// SIDE_EFFECTS: none
-// KEYWORDS:[PATTERN(Builder): immutable report envelope; CONCEPT(Contract): stable JSON]
-// LINKS:[USES_API(UUID): github.com/google/uuid; USES_API(Time): time.Now]
-// COMPLEXITY_SCORE:[3][Linear model assembly]
-// END_CONTRACT
 func NewReport(profile string, system SystemInfo, pg PostgreSQLInfo, checks []CheckResult) Report {
 	return Report{
 		SchemaVersion: SchemaVersion,
@@ -182,18 +145,6 @@ func NewReport(profile string, system SystemInfo, pg PostgreSQLInfo, checks []Ch
 	}
 }
 
-// START_FUNCTION_BuildSummary
-// START_CONTRACT:
-// PURPOSE:Aggregate summary counters from check statuses.
-// INPUTS:
-// - checks to aggregate => checks: []CheckResult
-// OUTPUTS:
-// - Summary - counts grouped by status classes
-// SIDE_EFFECTS: none
-// KEYWORDS:[PATTERN(Reduce): status aggregation; CONCEPT(Policy): exit-code groundwork]
-// LINKS:[USES_API(None): none]
-// COMPLEXITY_SCORE:[4][Single pass over checks]
-// END_CONTRACT
 func BuildSummary(checks []CheckResult) Summary {
 	s := Summary{Total: len(checks)}
 	for _, check := range checks {

internal/checks/io_scheduler.go

@@ -7,22 +7,6 @@ import (
 	"github.com/balyakin/pgkernel/internal/checker"
 )
 
-// FILE:internal/checks/io_scheduler.go
-// VERSION:1.0.0
-// START_MODULE_CONTRACT:
-// PURPOSE:Implement storage scheduler and dirty page write-back checks.
-// SCOPE:IO-001, IO-002.
-// INPUT:System storage and vm dirty ratio state.
-// OUTPUT:IO-focused recommendations for PostgreSQL reliability.
-// KEYWORDS:[DOMAIN(IO): scheduler; DOMAIN(MemoryWriteback): dirty ratios]
-// LINKS:[READS_DATA_FROM(/sys/block/*/queue/scheduler): scheduler]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.0 - Implemented IO checks.
-// PREV_CHANGE_SUMMARY:none
-// END_CHANGE_SUMMARY
-
 type io001Scheduler struct{}
 type io002DirtyRatios struct{}
 

internal/checks/kern_preemption.go

@@ -6,22 +6,6 @@ import (
 	"github.com/balyakin/pgkernel/internal/checker"
 )
 
-// FILE:internal/checks/kern_preemption.go
-// VERSION:1.0.0
-// START_MODULE_CONTRACT:
-// PURPOSE:Implement kernel preemption and kernel-version interaction checks.
-// SCOPE:KERN-001, KERN-002, KERN-003.
-// INPUT:checker.RuntimeState kernel attributes.
-// OUTPUT:Structured check results with remediation metadata.
-// KEYWORDS:[DOMAIN(Kernel): preemption model; DOMAIN(PostgreSQL): throughput risk]
-// LINKS:[READS_DATA_FROM(internal/detect/kernel.go): KernelState]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.0 - Implemented KERN check family according to specification.
-// PREV_CHANGE_SUMMARY:none
-// END_CHANGE_SUMMARY
-
 type kern001PreemptionModel struct{}
 type kern002KernelVersionRisk struct{}
 type kern003RSEQSupport struct{}

internal/checks/mem_hugepages.go

@@ -8,22 +8,6 @@ import (
 	"github.com/balyakin/pgkernel/internal/checker"
 )
 
-// FILE:internal/checks/mem_hugepages.go
-// VERSION:1.0.1
-// START_MODULE_CONTRACT:
-// PURPOSE:Implement huge pages and THP checks for PostgreSQL workloads.
-// SCOPE:MEM-001, MEM-002.
-// INPUT:System THP/hugepages and PostgreSQL settings.
-// OUTPUT:Actionable results with safety-aware remediation.
-// KEYWORDS:[DOMAIN(Memory): THP and HugePages; CONCEPT(Latency): compaction spikes]
-// LINKS:[READS_DATA_FROM(/sys/kernel/mm/transparent_hugepage/enabled): THP]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.1 - Fixed shared_buffers-to-hugepages estimate to use PostgreSQL unit semantics with dynamic huge page size.
-// PREV_CHANGE_SUMMARY:1.0.0 - Implemented memory hugepage checks.
-// END_CHANGE_SUMMARY
-
 type mem001THP struct{}
 type mem002StaticHugePages struct{}
 

internal/checks/mem_swap.go

@@ -6,22 +6,6 @@ import (
 	"github.com/balyakin/pgkernel/internal/checker"
 )
 
-// FILE:internal/checks/mem_swap.go
-// VERSION:1.0.0
-// START_MODULE_CONTRACT:
-// PURPOSE:Implement memory pressure and OOM sensitivity checks.
-// SCOPE:MEM-003, MEM-004, MEM-005.
-// INPUT:sysctl values and PostgreSQL process metadata.
-// OUTPUT:Structured memory risk assessment.
-// KEYWORDS:[DOMAIN(Memory): swappiness and overcommit; DOMAIN(Reliability): OOM resilience]
-// LINKS:[READS_DATA_FROM(/proc/sys/vm/*): sysctl]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.0 - Added swap and OOM checks.
-// PREV_CHANGE_SUMMARY:none
-// END_CHANGE_SUMMARY
-
 type mem003Swappiness struct{}
 type mem004Overcommit struct{}
 type mem005OOMAdj struct{}

internal/checks/net_tcp.go

@@ -6,22 +6,6 @@ import (
 	"github.com/balyakin/pgkernel/internal/checker"
 )
 
-// FILE:internal/checks/net_tcp.go
-// VERSION:1.0.0
-// START_MODULE_CONTRACT:
-// PURPOSE:Implement networking baseline checks relevant for remote PostgreSQL clients.
-// SCOPE:NET-001.
-// INPUT:TCP keepalive sysctl values.
-// OUTPUT:Informational or pass state with recommended tuning range.
-// KEYWORDS:[DOMAIN(Networking): tcp keepalive; CONCEPT(Resilience): stale connection detection]
-// LINKS:[READS_DATA_FROM(/proc/sys/net/ipv4/*): keepalive knobs]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.0 - Added TCP keepalive check.
-// PREV_CHANGE_SUMMARY:none
-// END_CHANGE_SUMMARY
-
 type net001TCPKeepalive struct{}
 
 func NetworkChecks() []checker.Check {

internal/checks/pg_config.go

@@ -8,22 +8,6 @@ import (
 	"github.com/balyakin/pgkernel/internal/checker"
 )
 
-// FILE:internal/checks/pg_config.go
-// VERSION:1.0.1
-// START_MODULE_CONTRACT:
-// PURPOSE:Implement PostgreSQL configuration sanity checks.
-// SCOPE:PG-001, PG-002, PG-003.
-// INPUT:Parsed postgresql.conf settings and system RAM.
-// OUTPUT:PostgreSQL tuning diagnostics aligned with host resources.
-// KEYWORDS:[DOMAIN(PostgreSQL): memory and WAL tuning; CONCEPT(Capacity): headroom]
-// LINKS:[READS_DATA_FROM(postgresql.conf): shared_buffers/work_mem/wal settings]
-// END_MODULE_CONTRACT
-
-// START_CHANGE_SUMMARY:
-// LAST_CHANGE:1.0.1 - Fixed PostgreSQL memory unit normalization for shared_buffers/work_mem/wal_buffers calculations.
-// PREV_CHANGE_SUMMARY:1.0.0 - Implemented PostgreSQL sanity checks.
-// END_CHANGE_SUMMARY
-
 type pg001SharedBuffers struct{}
 type pg002WorkMem struct{}
 type pg003WAL struct{}