From 094f032af033f07f9719ef4acf560302527e1e43 Mon Sep 17 00:00:00 2001
From: Cristian Magherusan-Stanciu <cristi.magherusan@gmail.com>
Date: Tue, 20 Oct 2026 19:31:49 +0200
Subject: [PATCH] chore(lint): drop monorepo path exclusions from .golangci.yml

Removes the exclusions for internal/, pkg/ and cmd/configure_* paths and generated files, none of which exist in this repo. make lint still reports 0 issues. Part of #2 (item 7; .gitallowed and .gitleaks.toml already carry no internal/ entries).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .golangci.yml | 28 ----------------------------
 1 file changed, 28 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index ed78d9d..281dc77 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -91,40 +91,12 @@ linters:
           - govet
         text: "unusedwrite"
         path: _test\.go
-      - linters:
-          - errcheck
-          - gosec
-        path: internal/testutil/
-      - linters:
-          - all
-        path: .*_gen\.go
-      # revive var-naming: "api" is an intentional package name for the HTTP API
-      # layer; it is not a utility package and "avoid meaningless package names"
-      # is a false positive here. All 88+ files in internal/api use this name.
-      - linters:
-          - revive
-        path: internal/api/
-        text: "avoid meaningless package names"
-      # noctx: exec.Command calls in configure CLI helpers intentionally
-      # omit context (CommandRunner interface contract; operator-controlled inputs).
-      # Suppressed per-file rather than per-line so #nosec G204 can lead the comment.
-      - linters:
-          - noctx
-        path: (cmd/configure_gcp|cmd/configure_azure)\.go
       # noctx: http.Get and net.Listen in test helpers appropriately use the
       # background context; threading a test context through every helper would
       # add noise without improving test correctness.
       - linters:
           - noctx
         path: _test\.go
-      # misspell: "cancelled" is the canonical DB column name (migration 000035,
-      # UK spelling throughout; migration #1277 tracks the rename). "initialised"
-      # appears in established comments in internal/config/types.go matching the
-      # existing UK convention. Allow both spellings in the files listed below.
-      - linters:
-          - misspell
-        path: (internal/config/(store_postgres|store_postgres_pgxmock_test|store_postgres_ladder|types|interfaces)|pkg/ladder/(store|types_test)|internal/api/(handler_history|handler_ri_exchange(_test)?|router_handlers_test|coverage_extras_test)|internal/mocks/stores|internal/purchase/scheduled_fire_test)\.go
-        text: "cancelled|initialised"
     paths:
       - third_party$
       - builtin$