Refactor (#31)

* Update golangci-lint configuration and add project rules documentation

* Add golangci-lint configuration file

* Add .serena to .gitignore

* Update revive linter rules in golangci configuration

* Enhance test parallelism and update documentation

- Added `t.Parallel()` to all test functions across various test files to improve test execution speed by allowing concurrent test execution.
- Updated AGENTS.md to include usage instructions for golangci-lint.

* refactor(tests): replace assert with require for error handling in test cases

- Updated multiple test files to use `require.NoError` and `require.Error` instead of `assert.NoError` and `assert.Error` for better error handling.
- Ensured consistency across test cases in `delete_test.go`, `expr_test.go`, `insert_test.go`, `placeholder_test.go`, `select_test.go`, `statement_test.go`, `update_test.go`, and `where_test.go`.
- Improved readability and maintainability of tests by using `require` assertions.

* Refactor SQL builder methods for consistency and error handling

- Updated error handling in insert, update, and select builders to use more concise error messages.
- Changed assertions in tests from assert to require for better failure handling.
- Refactored SQL generation methods to return named return values for clarity.
- Improved documentation comments for various methods to enhance code readability.
- Adjusted placeholder handling in SQL generation to ensure proper formatting.
- Enhanced pagination logic in the select builder to handle edge cases more effectively.

* refactor: update init functions to include linter directives for builder registrations

* refactor: streamline SQL generation methods for better readability and maintainability

* refactor(tests): add tests for REPLACE and DELETE queries with CTEs

Author: n-r-w

.gitignore

@@ -0,0 +1 @@
+/.serena

.golangci.yaml

@@ -0,0 +1,156 @@
+run:
+  timeout: 5m
+  issues-exit-code: 1
+  tests: true
+
+version: 2
+
+output:
+  format: colored-line-number
+  print-issued-lines: true
+  print-linter-name: true
+
+linters:
+  disable-all: true
+  enable:
+    - godot
+    - copyloopvar
+    - gocyclo
+    - misspell
+    - rowserrcheck
+    - staticcheck
+    - asasalint
+    - asciicheck
+    - bidichk
+    - bodyclose
+    # - cyclop
+    - dupl
+    - durationcheck
+    - errcheck
+    - errname
+    - errorlint
+    - exhaustive
+    - exhaustruct
+    - forbidigo
+    - gocheckcompilerdirectives
+    - gochecknoglobals
+    - gochecknoinits
+    - gochecksumtype
+    - gocognit
+    - goconst
+    - gocritic
+    - gomoddirectives
+    - gomodguard
+    - goprintffuncname
+    - gosec
+    - govet
+    - ineffassign
+    - lll
+    - loggercheck
+    - makezero
+    - mirror
+    - mnd
+    - musttag
+    - nakedret
+    - nestif
+    - nilerr
+    - nilnil
+    - noctx
+    - nolintlint
+    - nosprintfhostport
+    - perfsprint
+    - prealloc
+    - predeclared
+    - promlinter
+    - protogetter
+    - reassign
+    - revive
+    - sloglint
+    - sqlclosecheck
+    - testableexamples
+    - testifylint
+    - tparallel
+    - unconvert
+    - unparam
+    - unused
+    - usestdlibvars
+    - wastedassign
+    - whitespace
+    - paralleltest
+  settings:
+    revive:
+      rules:
+        - name: var-naming
+          disabled: true
+    govet:
+      check-shadowing: true
+    gocyclo:
+      min-complexity: 15
+    dupl:
+      threshold: 100
+    goconst:
+      min-len: 2
+      min-occurrences: 2
+    misspell:
+      locale: US
+    lll:
+      line-length: 120
+    goimports:
+      local-prefixes: github.com/n-r-w/squirrel
+    nolintlint:
+      require-explanation: true
+      require-specific: true
+    gocritic:
+      enabled-tags:
+        - diagnostic
+        - experimental
+        - opinionated
+        - performance
+        - style
+      disabled-checks:
+        - dupImport
+        - ifElseChain
+        - octalLiteral
+        - whyNoLint
+        - wrapperFunc
+  exclusions:
+    generated: lax
+    rules:
+      - linters:
+          - perfsprint
+          - dupl
+          - gochecknoglobals
+          - gocognit
+          - goconst
+          - gocritic
+          - gosec
+          - govet
+          - lll
+          - revive
+          - staticcheck
+          - unparam
+          - whitespace
+          - gocyclo
+          - cyclop
+        path: _test\.go
+
+  exclude-use-default: false
+  max-issues-per-linter: 0
+  max-same-issues: 0
+  new: false
+
+severity:
+  default-severity: error
+  case-sensitive: false
+
+formatters:
+  enable:
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - loadtest$
+      - profile$
+      - bench$

.golangci.yml

@@ -0,0 +1,7 @@
+# Project Rules
+
+## Tech Stack
+
+- go 1.18
+- github.com/stretchr/testify for testing
+- golangci-lint-v2 for linting (`golangci-lint run --config .golangci.yml ./... `)