refactor(internal): move private packages under internal

README.md

@@ -5,6 +5,10 @@
 
 `prec` is a Linux command execution observability tool where `precd` uses eBPF to continuously collect process execution events as JSON Lines and `prec` lets you quickly search and inspect those records from the CLI.
 
+Implementation note:
+
+- Go packages under `internal/` are private implementation details and are not a supported public import API
+
 ## Why use it
 
 - Does not rely on shell history or `LD_PRELOAD`

cmd/prec/main.go

@@ -7,7 +7,7 @@ import (
 	"fmt"
 	"os"
 
-	"github.com/jfut/prec/pkg/cli"
+	"github.com/jfut/prec/internal/cli"
 )
 
 // Build metadata fields are injected by linker flags at build time.

cmd/precd/main.go

@@ -13,9 +13,9 @@ import (
 
 	"github.com/alecthomas/kong"
 
-	"github.com/jfut/prec/pkg/collector"
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/logger"
+	"github.com/jfut/prec/internal/collector"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/logger"
 )
 
 // Build metadata fields are injected by linker flags at build time.

pkg/cli/app.go → internal/cli/app.go

@@ -11,8 +11,8 @@ import (
 
 	"github.com/alecthomas/kong"
 
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/query"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/query"
 )
 
 type listFilter struct {

pkg/cli/app_test.go → internal/cli/app_test.go

@@ -15,7 +15,7 @@ import (
 
 	"github.com/alecthomas/kong"
 
-	"github.com/jfut/prec/pkg/config"
+	"github.com/jfut/prec/internal/config"
 )
 
 func TestValidateModeFlags(t *testing.T) {

pkg/cli/event_collection.go → internal/cli/event_collection.go

@@ -15,7 +15,7 @@ import (
 	"github.com/klauspost/compress/gzip"
 	"github.com/klauspost/compress/zstd"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 func collectFilteredEvents(logPaths []string, limit int, lf listFilter) ([]events.CommandEvent, error) {

pkg/cli/event_collection_test.go → internal/cli/event_collection_test.go

@@ -7,7 +7,7 @@ import (
 	"path/filepath"
 	"testing"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 func TestCollectFilteredEventsAcrossRotatedLogs(t *testing.T) {

pkg/cli/event_merge.go → internal/cli/event_merge.go

@@ -3,7 +3,7 @@
 
 package cli
 
-import "github.com/jfut/prec/pkg/events"
+import "github.com/jfut/prec/internal/events"
 
 type commandJoinState struct {
 	pending map[string]events.CommandEvent

pkg/cli/event_merge_test.go → internal/cli/event_merge_test.go

@@ -6,7 +6,7 @@ package cli
 import (
 	"testing"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 func TestMergeEventsForListJoinsStartEnd(t *testing.T) {

pkg/cli/log_reader_test.go → internal/cli/log_reader_test.go

@@ -10,8 +10,8 @@ import (
 	"sync"
 	"testing"
 
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/events"
 )
 
 func TestOpenLogReaderDetectsCompressionByMagic(t *testing.T) {

pkg/cli/output_formatter.go → internal/cli/output_formatter.go

@@ -13,7 +13,7 @@ import (
 	"strings"
 	"time"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 // outputFormatter hides per-format encoding details from list/follow flows.

pkg/cli/output_formatter_test.go → internal/cli/output_formatter_test.go

@@ -8,7 +8,7 @@ import (
 	"strings"
 	"testing"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 func TestFormatEventSimple(t *testing.T) {

pkg/cli/query_filter.go → internal/cli/query_filter.go

@@ -4,8 +4,8 @@
 package cli
 
 import (
-	"github.com/jfut/prec/pkg/events"
-	"github.com/jfut/prec/pkg/query"
+	"github.com/jfut/prec/internal/events"
+	"github.com/jfut/prec/internal/query"
 )
 
 func buildQueryFilter(specs []string) (listFilter, error) {

pkg/cli/query_filter_test.go → internal/cli/query_filter_test.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 func TestBuildQueryFilterValid(t *testing.T) {

pkg/cli/tail.go → internal/cli/tail.go

@@ -13,7 +13,7 @@ import (
 	"strings"
 	"time"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 func runList(logPaths []string, limit int, lf listFilter, opt outputOptions) int {

pkg/cli/test_helpers_test.go → internal/cli/test_helpers_test.go

@@ -15,7 +15,7 @@ import (
 
 	"github.com/klauspost/compress/zstd"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 func intPtr(v int) *int {

pkg/collector/collector_linux.go → internal/collector/collector_linux.go

@@ -24,10 +24,10 @@ import (
 	"github.com/cilium/ebpf/rlimit"
 	"golang.org/x/sys/unix"
 
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/events"
-	"github.com/jfut/prec/pkg/filter"
-	"github.com/jfut/prec/pkg/logger"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/events"
+	"github.com/jfut/prec/internal/filter"
+	"github.com/jfut/prec/internal/logger"
 )
 
 const (

pkg/collector/collector_linux_test.go → internal/collector/collector_linux_test.go

@@ -16,9 +16,9 @@ import (
 	"testing"
 
 	"github.com/cilium/ebpf/asm"
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/events"
-	"github.com/jfut/prec/pkg/logger"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/events"
+	"github.com/jfut/prec/internal/logger"
 )
 
 func TestParseOffsetFromFormat(t *testing.T) {

pkg/filter/filter.go → internal/filter/filter.go

@@ -7,9 +7,9 @@ import (
 	"fmt"
 	"strings"
 
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/events"
-	"github.com/jfut/prec/pkg/query"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/events"
+	"github.com/jfut/prec/internal/query"
 )
 
 type compiledRule struct {

pkg/filter/filter_test.go → internal/filter/filter_test.go

@@ -6,8 +6,8 @@ package filter
 import (
 	"testing"
 
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/events"
 )
 
 func TestMatcherRulesFirstMatchWins(t *testing.T) {

pkg/logger/jsonl.go → internal/logger/jsonl.go

@@ -16,8 +16,8 @@ import (
 	"github.com/klauspost/compress/gzip"
 	"github.com/klauspost/compress/zstd"
 
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/events"
 )
 
 // JSONLWriter appends events line by line to a root-owned log file.

pkg/logger/jsonl_test.go → internal/logger/jsonl_test.go

@@ -15,8 +15,8 @@ import (
 	"github.com/klauspost/compress/gzip"
 	"github.com/klauspost/compress/zstd"
 
-	"github.com/jfut/prec/pkg/config"
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/config"
+	"github.com/jfut/prec/internal/events"
 )
 
 func TestJSONLWriterReopenSwitchesOutputFile(t *testing.T) {

pkg/query/filter.go → internal/query/filter.go

@@ -12,7 +12,7 @@ import (
 	"sync"
 	"time"
 
-	"github.com/jfut/prec/pkg/events"
+	"github.com/jfut/prec/internal/events"
 )
 
 var operators = []string{"!~=", ">=", "<=", "!=", "~=", ">", "<", "="}

justfile

@@ -45,7 +45,7 @@ deps:
 #
 
 fmt:
-    gofmt -w ./cmd ./pkg
+    gofmt -w .
 
 lint:
     golangci-lint run ./...