Refactor: Move packages to pkg/

api/schemas/main_test.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap/zapcore"
 )
 

cmd/cmd_test.go

@@ -10,7 +10,7 @@ import (
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // TestConfigFlagOverride verifies that CLI flags correctly override configuration

cmd/evolution.go

@@ -11,7 +11,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/evolution/analyst"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"github.com/xkilldash9x/scalpel-cli/internal/service"
 )
 

cmd/evolution_test.go

@@ -13,7 +13,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/mocks"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // MockAnalystRunner is a mock implementation of the AnalystRunner interface.

cmd/main_test.go

@@ -8,7 +8,7 @@ import (
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"github.com/xkilldash9x/scalpel-cli/internal/service" // FIX: Import the service package
 	"go.uber.org/zap"
 )

cmd/report.go

@@ -14,7 +14,7 @@ import (
 
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"github.com/xkilldash9x/scalpel-cli/internal/reporting"
 	"github.com/xkilldash9x/scalpel-cli/internal/results"
 	"github.com/xkilldash9x/scalpel-cli/internal/store"

cmd/root.go

@@ -11,7 +11,7 @@ import (
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"github.com/xkilldash9x/scalpel-cli/internal/service"
 	"go.uber.org/zap"
 )

cmd/scalpel/main.go

@@ -19,7 +19,7 @@ import (
 	"time"
 
 	"github.com/xkilldash9x/scalpel-cli/cmd"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 const panicLogFile = "panic.log"

cmd/scan.go

@@ -18,7 +18,7 @@ import (
 
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"github.com/xkilldash9x/scalpel-cli/internal/reporting"
 	"github.com/xkilldash9x/scalpel-cli/internal/results"
 	"github.com/xkilldash9x/scalpel-cli/internal/service"

cmd/scan_test.go

@@ -16,7 +16,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/mocks"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"github.com/xkilldash9x/scalpel-cli/internal/service"
 	"go.uber.org/zap/zapcore"
 )

cmd/self_heal.go

@@ -10,7 +10,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/autofix/metalyst"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"github.com/xkilldash9x/scalpel-cli/internal/service"
 	"go.uber.org/zap"
 )

cmd/self_heal_test.go

@@ -12,7 +12,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/mocks"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // MockMetalystRunner is a mock implementation of the MetalystRunner interface.

go.mod

@@ -3,7 +3,6 @@ module github.com/xkilldash9x/scalpel-cli
 go 1.25.4
 
 require (
-	github.com/AdaLogics/go-fuzz-headers v0.0.0-20240806141605-e8a1dd7889d6
 	github.com/andybalholm/brotli v1.2.0
 	github.com/beevik/etree v1.6.0
 	github.com/cespare/xxhash/v2 v2.3.0

go.sum

@@ -6,8 +6,6 @@ cloud.google.com/go/compute/metadata v0.9.0 h1:pDUj4QMoPejqq20dK0Pg2N4yG9zIkYGdB
 cloud.google.com/go/compute/metadata v0.9.0/go.mod h1:E0bWwX5wTnLPedCKqk3pJmVgCBSM6qQI1yTBdEb3C10=
 dario.cat/mergo v1.0.2 h1:85+piFYR1tMbRrLcDwR18y4UKJ3aH1Tbzi24VRW1TK8=
 dario.cat/mergo v1.0.2/go.mod h1:E/hbnu0NxMFBjpMIE34DRGLWqDy0g5FuKDhCb31ngxA=
-github.com/AdaLogics/go-fuzz-headers v0.0.0-20240806141605-e8a1dd7889d6 h1:He8afgbRMd7mFxO99hRNu+6tazq8nFF9lIwo9JFroBk=
-github.com/AdaLogics/go-fuzz-headers v0.0.0-20240806141605-e8a1dd7889d6/go.mod h1:8o94RPi1/7XTJvwPpRSzSUedZrtlirdB3r9Z20bi2f8=
 github.com/Microsoft/go-winio v0.5.2/go.mod h1:WpS1mjBmmwHBEWmogvA2mj8546UReBk4v8QkMxJ6pZY=
 github.com/Microsoft/go-winio v0.6.2 h1:F2VQgta7ecxGYO8k3ZZz3RS8fVIXVxONVUPlNERoyfY=
 github.com/Microsoft/go-winio v0.6.2/go.mod h1:yd8OoFMLzJbo9gZq8j5qaps8bJ9aShtEA8Ipt1oGCvU=

internal/agent/analysis_executor.go

@@ -8,7 +8,7 @@ import (
 	"net/url"
 	"time"
 
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"

internal/agent/analysis_executor_test.go

@@ -15,7 +15,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/analysis/core"
 	"github.com/xkilldash9x/scalpel-cli/internal/mocks"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // Setup function for AnalysisExecutor tests.

internal/agent/codebase_executor.go

@@ -12,7 +12,7 @@ import (
 	"path/filepath"
 	"sort"
 
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 	"golang.org/x/tools/go/packages"
 )

internal/agent/executors.go

@@ -11,7 +11,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/analysis/core"
 	"github.com/xkilldash9x/scalpel-cli/internal/browser/humanoid"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/agent/humanoid_executor.go

@@ -8,7 +8,7 @@ import (
 
 	json "github.com/json-iterator/go"
 	"github.com/xkilldash9x/scalpel-cli/internal/browser/humanoid"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/agent/humanoid_executor_test.go

@@ -13,7 +13,7 @@ import (
 
 	"github.com/xkilldash9x/scalpel-cli/internal/browser/humanoid"
 	"github.com/xkilldash9x/scalpel-cli/internal/mocks"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // setupHumanoidExecutorTest initializes the executor with a mock humanoid controller.

internal/agent/login_executor.go

@@ -9,7 +9,7 @@ import (
 
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/browser/humanoid"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/agent/main_test.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap/zapcore"
 )
 

internal/agent/signup_executor.go

@@ -22,7 +22,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/browser/humanoid"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/agent/signup_executor_test.go

@@ -21,7 +21,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/internal/browser/humanoid"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/mocks"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 	"go.uber.org/zap/zaptest"
 )

internal/analysis/active/timeslip/concurrency_test.go

@@ -17,7 +17,7 @@ import (
 	"golang.org/x/net/http2"
 
 	// -- ADDED --
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // --- III. Concurrency & Reliability Tests ---

internal/analysis/active/timeslip/e2e_test.go

@@ -18,7 +18,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/internal/analysis/active/timeslip"
 	"github.com/xkilldash9x/scalpel-cli/internal/analysis/core"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // MockReporter for E2E tests (Thread-safe).

internal/analysis/active/timeslip/graphql_async.go

@@ -10,7 +10,7 @@ import (
 	"net/http"
 	"time"
 
-	"github.com/xkilldash9x/scalpel-cli/internal/browser/network"
+	"github.com/xkilldash9x/scalpel-cli/pkg/network"
 	"go.uber.org/zap"
 )
 

internal/analysis/active/timeslip/h1_concurrent.go

@@ -10,7 +10,7 @@ import (
 	"sync"
 	"time"
 
-	"github.com/xkilldash9x/scalpel-cli/internal/browser/network/customhttp"
+	"github.com/xkilldash9x/scalpel-cli/pkg/customhttp"
 	"go.uber.org/zap"
 )
 

internal/analysis/active/timeslip/h1_singlebyte.go

@@ -12,7 +12,7 @@ import (
 	"net/url"
 	"time"
 
-	"github.com/xkilldash9x/scalpel-cli/internal/browser/network"
+	"github.com/xkilldash9x/scalpel-cli/pkg/network"
 	"go.uber.org/zap"
 )
 

internal/analysis/active/timeslip/h1_singlebyte_test.go

@@ -11,7 +11,7 @@ import (
 
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
-	"github.com/xkilldash9x/scalpel-cli/internal/browser/network"
+	"github.com/xkilldash9x/scalpel-cli/pkg/network"
 )
 
 // TestSetupConnectionDetails validates the configuration of the DialerConfig

internal/analysis/active/timeslip/h2_multiplex.go

@@ -11,7 +11,7 @@ import (
 	"sync"
 	"time"
 
-	"github.com/xkilldash9x/scalpel-cli/internal/browser/network/customhttp"
+	"github.com/xkilldash9x/scalpel-cli/pkg/customhttp"
 	"go.uber.org/zap"
 )
 

internal/analysis/active/timeslip/integration_test.go

@@ -22,7 +22,7 @@ import (
 	"github.com/stretchr/testify/require"
 
 	// -- ADDED --
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // Helper struct to track requests received by the mock server in a thread-safe manner.

internal/analysis/active/timeslip/main_test.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap/zapcore"
 )
 

internal/analysis/active/timeslip/timeslip_analyzer.go

@@ -16,7 +16,7 @@ import (
 	"github.com/google/uuid"
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
 	"github.com/xkilldash9x/scalpel-cli/internal/analysis/core"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/analysis/auth/idor/analyzer_test.go

@@ -14,7 +14,7 @@ import (
 
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
-	"github.com/xkilldash9x/scalpel-cli/internal/browser/network"
+	"github.com/xkilldash9x/scalpel-cli/pkg/network"
 	"github.com/xkilldash9x/scalpel-cli/internal/jsoncompare"
 )
 

internal/analysis/auth/idor/idor_analyzer.go

@@ -7,9 +7,9 @@ import (
 	"log"
 	"runtime"
 
-	"github.com/xkilldash9x/scalpel-cli/internal/browser/network"
+	"github.com/xkilldash9x/scalpel-cli/pkg/network"
 	"github.com/xkilldash9x/scalpel-cli/internal/jsoncompare"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 )
 
 // IDORAnalyzer is the main orchestrator for the Insecure Direct Object Reference

internal/analysis/auth/idor/idor_test.go

@@ -20,7 +20,7 @@ import (
 	"github.com/stretchr/testify/require"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/jsoncompare"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/goleak"
 )
 

internal/autofix/metalyst/matalyst_test.go

@@ -18,7 +18,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/internal/autofix/coroner"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/mocks"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap/zapcore"
 	"go.uber.org/zap/zaptest"
 )

internal/autofix/metalyst/metalyst.go

@@ -18,7 +18,7 @@ import (
 	"github.com/xkilldash9x/scalpel-cli/internal/autofix/coroner"
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
 	"github.com/xkilldash9x/scalpel-cli/internal/llmutil"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/discovery/adapter.go

@@ -7,7 +7,7 @@ import (
 	"net/http"
 
 	// This import is no longer needed since we are using the standard http.Client.
-	// "github.com/xkilldash9x/scalpel-cli/internal/browser/network"
+	// "github.com/xkilldash9x/scalpel-cli/pkg/network"
 )
 
 // HTTPClient interface (defined elsewhere in discovery package, e.g., passive.go):

internal/discovery/discover_dom.go

@@ -8,7 +8,7 @@ import (
 	"sync"
 
 	"github.com/xkilldash9x/scalpel-cli/api/schemas"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/jsoncompare/main_test.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap/zapcore"
 )
 

internal/jsoncompare/service.go

@@ -14,7 +14,7 @@ import (
 	"github.com/google/go-cmp/cmp"
 	"github.com/google/go-cmp/cmp/cmpopts"
 	"github.com/google/uuid"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap"
 )
 

internal/reporting/main_test.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 
 	"github.com/xkilldash9x/scalpel-cli/internal/config"
-	"github.com/xkilldash9x/scalpel-cli/internal/observability"
+	"github.com/xkilldash9x/scalpel-cli/pkg/observability"
 	"go.uber.org/zap/zapcore"
 )