modified: .gitignore

	new file:   api/schemas/main_test.go
	modified:   internal/agent/main_test.go
	modified:   internal/analysis/active/taint/taint_shim.js
	modified:   internal/analysis/active/taint/taint_shim.test.js
	modified:   internal/analysis/active/timeslip/main_test.go
	modified:   internal/browser/stealth/evasions.js
	modified:   internal/browser/stealth/evasions.test.js
	modified:   internal/browser/stealth/js_test.go
	modified:   internal/config/config.go
	modified:   internal/jsoncompare/main_test.go
	modified:   internal/knowledgegraph/postgres_kg.go
	modified:   internal/observability/logger_test.go
	new file:   internal/observability/main_test.go
	new file:   internal/reporting/main_test.go
	modified:   internal/reporting/reporter.go
	modified:   internal/reporting/reporter_test.go
	modified:   internal/reporting/sarif_reporter.go
	modified:   internal/reporting/sarif_reporter_test.go
	modified:   package-lock.json

Author: xkilldash9x

.gitignore

@@ -22,11 +22,11 @@ logs/
 # Local configuration overrides and environment files
 # It's good practice to commit config.yaml as a template,
 # but ignore local versions with secrets.
+config.yaml
 *.local.yaml
 .credentials.json
 .env
 .env.*
-config.yaml
 # Corrected path based on project structure
 internal/browser/testing/
 test_imports.go
@@ -59,6 +59,5 @@ Thumbs.db
 *.tar.gz
 *.rar
 .envrc
-scalpel
 node_modules/
 config.yaml

api/schemas/main_test.go

@@ -0,0 +1,40 @@
+
+// File: api/schemas/main_test.go
+package schemas_test
+
+import (
+	"os"
+	"testing"
+
+	"github.com/xkilldash9x/scalpel-cli/internal/config"
+	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"go.uber.org/zap/zapcore"
+)
+
+// TestMain serves as the entry point for all tests in the schemas package.
+// It instantiates the global dependency-injected logger before running tests.
+func TestMain(m *testing.M) {
+	// 1. Load default configuration.
+	appConfig := config.NewDefaultConfig()
+	logConfig := appConfig.Logger()
+
+	// 2. Override settings for the test environment.
+	logConfig.Level = "debug"
+	logConfig.ServiceName = "test-suite"
+	logConfig.Format = "console"
+
+	// 3. Initialize the global logger.
+	observability.Initialize(logConfig, zapcore.Lock(os.Stdout))
+
+	// 4. Run the tests.
+	exitCode := m.Run()
+
+	// 5. Teardown and Sync.
+	observability.Sync()
+
+	// Explicitly reset the global state to be clean.
+	observability.ResetForTest()
+
+	// 6. Exit with the result code.
+	os.Exit(exitCode)
+}

internal/agent/main_test.go

@@ -1,23 +1,40 @@
-package agent
+
+// File: internal/agent/main_test.go
+package agent_test
 
 import (
 	"os"
 	"testing"
 
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"go.uber.org/zap/zapcore"
 )
 
-// TestMain is executed before any tests in this package.
+// TestMain serves as the entry point for all tests in the agent package.
+// It instantiates the global dependency-injected logger before running tests.
 func TestMain(m *testing.M) {
-	// Initialize a simple logger for testing purposes to avoid spamming the console.
-	// This prevents the "Global logger requested before initialization" warning.
-	cfg := config.NewDefaultConfig().Logger()
-	cfg.Level = "fatal" // Silence all logs except fatal during tests
-	observability.InitializeLogger(cfg)
+	// 1. Load default configuration.
+	appConfig := config.NewDefaultConfig()
+	logConfig := appConfig.Logger()
+
+	// 2. Override settings for the test environment.
+	logConfig.Level = "debug"
+	logConfig.ServiceName = "test-suite"
+	logConfig.Format = "console"
+
+	// 3. Initialize the global logger.
+	observability.Initialize(logConfig, zapcore.Lock(os.Stdout))
+
+	// 4. Run the tests.
+	exitCode := m.Run()
+
+	// 5. Teardown and Sync.
+	observability.Sync()
 
-	// After running all tests, ensure logs are flushed.
-	defer observability.Sync()
+	// Explicitly reset the global state to be clean.
+	observability.ResetForTest()
 
-	os.Exit(m.Run())
+	// 6. Exit with the result code.
+	os.Exit(exitCode)
 }