AdGuardHome/scripts/make/go-lint.sh

#!/bin/sh

# This comment is used to simplify checking local copies of the script.  Bump
# this number every time a significant change is made to this script.
#
# AdGuard-Project-Version: 5

verbose="${VERBOSE:-0}"
readonly verbose

if [ "$verbose" -gt '0' ]
then
	set -x
fi

# Set $EXIT_ON_ERROR to zero to see all errors.
if [ "${EXIT_ON_ERROR:-1}" -eq '0' ]
then
	set +e
else
	set -e
fi

set -f -u



# Source the common helpers, including not_found and run_linter.
. ./scripts/make/helper.sh



# Simple analyzers

# blocklist_imports is a simple check against unwanted packages.  The following
# packages are banned:
#
#   *  Packages errors and log are replaced by our own packages in the
#      github.com/AdguardTeam/golibs module.
#
#   *  Package io/ioutil is soft-deprecated.
#
#   *  Package reflect is often an overkill, and for deep comparisons there are
#      much better functions in module github.com/google/go-cmp.  Which is
#      already our indirect dependency and which may or may not enter the stdlib
#      at some point.
#
#      See https://github.com/golang/go/issues/45200.
#
#   *  Package sort is replaced by package slices.
#
#   *  Package unsafe is… unsafe.
#
#   *  Package golang.org/x/exp/slices has been moved into stdlib.
#
#   *  Package golang.org/x/net/context has been moved into stdlib.
#
# Currently, the only standard exception are files generated from protobuf
# schemas, which use package reflect.  If your project needs more exceptions,
# add and document them.
#
# TODO(a.garipov): Add golibs/log.
#
# TODO(a.garipov): Add deprecated package golang.org/x/exp/maps once all
# projects switch to Go 1.22.
blocklist_imports() {
	git grep\
		-e '[[:space:]]"errors"$'\
		-e '[[:space:]]"io/ioutil"$'\
		-e '[[:space:]]"log"$'\
		-e '[[:space:]]"reflect"$'\
		-e '[[:space:]]"sort"$'\
		-e '[[:space:]]"unsafe"$'\
		-e '[[:space:]]"golang.org/x/exp/slices"$'\
		-e '[[:space:]]"golang.org/x/net/context"$'\
		-n\
		-- '*.go'\
		':!*.pb.go'\
		| sed -e 's/^\([^[:space:]]\+\)\(.*\)$/\1 blocked import:\2/'\
		|| exit 0
}

# method_const is a simple check against the usage of some raw strings and
# numbers where one should use named constants.
method_const() {
	git grep -F\
		-e '"DELETE"'\
		-e '"GET"'\
		-e '"PATCH"'\
		-e '"POST"'\
		-e '"PUT"'\
		-n\
		-- '*.go'\
		| sed -e 's/^\([^[:space:]]\+\)\(.*\)$/\1 http method literal:\2/'\
		|| exit 0
}

# underscores is a simple check against Go filenames with underscores.  Add new
# build tags and OS as you go.  The main goal of this check is to discourage the
# use of filenames like client_manager.go.
underscores() {
	underscore_files="$(
		git ls-files '*_*.go'\
			| grep -F\
			-e '_bsd.go'\
			-e '_darwin.go'\
			-e '_freebsd.go'\
			-e '_linux.go'\
			-e '_next.go'\
			-e '_openbsd.go'\
			-e '_others.go'\
			-e '_test.go'\
			-e '_unix.go'\
			-e '_windows.go'\
			-v\
			| sed -e 's/./\t\0/'
	)"
	readonly underscore_files

	if [ "$underscore_files" != '' ]
	then
		echo 'found file names with underscores:'
		echo "$underscore_files"
	fi
}

# TODO(a.garipov): Add an analyzer to look for `fallthrough`, `goto`, and `new`?



# Checks

run_linter -e blocklist_imports

run_linter -e method_const

run_linter -e underscores

run_linter -e gofumpt --extra -e -l .

# TODO(a.garipov): golint is deprecated, find a suitable replacement.

run_linter "$GO" vet ./...

run_linter govulncheck ./...

run_linter gocyclo --over 10 .

# TODO(a.garipov): Enable 10 for all.
run_linter gocognit --over='20'\
	./internal/querylog/\
	;

run_linter gocognit --over='19'\
	./internal/home/\
	;

run_linter gocognit --over='18'\
	./internal/aghtls/\
	;

run_linter gocognit --over='15'\
	./internal/aghos/\
	./internal/filtering/\
	;

run_linter gocognit --over='14'\
	./internal/dhcpd\
	;

run_linter gocognit --over='13'\
	./internal/aghnet/\
	;

run_linter gocognit --over='12'\
	./internal/filtering/rewrite/\
	;

run_linter gocognit --over='11'\
	./internal/updater/\
	;

run_linter gocognit --over='10'\
	./internal/aghalg/\
	./internal/aghhttp/\
	./internal/aghrenameio/\
	./internal/aghtest/\
	./internal/arpdb/\
	./internal/client/\
	./internal/configmigrate/\
	./internal/dhcpsvc\
	./internal/dnsforward/\
	./internal/filtering/hashprefix/\
	./internal/filtering/rulelist/\
	./internal/filtering/safesearch/\
	./internal/ipset\
	./internal/next/\
	./internal/rdns/\
	./internal/schedule/\
	./internal/stats/\
	./internal/tools/\
	./internal/version/\
	./internal/whois/\
	./scripts/\
	;

run_linter ineffassign ./...

run_linter unparam ./...

git ls-files -- 'Makefile' '*.conf' '*.go' '*.mod' '*.sh' '*.yaml' '*.yml'\
	| xargs misspell --error\
	| sed -e 's/^/misspell: /'

run_linter looppointer ./...

run_linter nilness ./...

# TODO(a.garipov): Enable for all.
run_linter fieldalignment \
	./internal/aghalg/\
	./internal/aghhttp/\
	./internal/aghos/\
	./internal/aghrenameio/\
	./internal/aghtest/\
	./internal/aghtls/\
	./internal/arpdb/\
	./internal/client/\
	./internal/configmigrate/\
	./internal/dhcpsvc/\
	./internal/filtering/hashprefix/\
	./internal/filtering/rewrite/\
	./internal/filtering/rulelist/\
	./internal/filtering/safesearch/\
	./internal/ipset/\
	./internal/next/...\
	./internal/querylog/\
	./internal/rdns/\
	./internal/schedule/\
	./internal/stats/\
	./internal/updater/\
	./internal/version/\
	./internal/whois/\
	;

run_linter -e shadow --strict ./...

# TODO(a.garipov): Enable for all.
run_linter gosec --quiet\
	./internal/aghalg/\
	./internal/aghchan/\
	./internal/aghhttp/\
	./internal/aghnet/\
	./internal/aghos/\
	./internal/aghrenameio/\
	./internal/aghtest/\
	./internal/arpdb/\
	./internal/client/\
	./internal/configmigrate/\
	./internal/dhcpd/\
	./internal/dhcpsvc/\
	./internal/dnsforward/\
	./internal/filtering/hashprefix/\
	./internal/filtering/rewrite/\
	./internal/filtering/rulelist/\
	./internal/filtering/safesearch/\
	./internal/ipset/\
	./internal/next/\
	./internal/rdns/\
	./internal/schedule/\
	./internal/stats/\
	./internal/tools/\
	./internal/version/\
	./internal/whois/\
	;

run_linter errcheck ./...

staticcheck_matrix='
darwin:  GOOS=darwin
freebsd: GOOS=freebsd
linux:   GOOS=linux
openbsd: GOOS=openbsd
windows: GOOS=windows
'
readonly staticcheck_matrix

echo "$staticcheck_matrix" | run_linter staticcheck --matrix ./...