clerk: internal/linter/rule_duplicated

clerk/internal/linter/rule_duplicated_account.go (view raw)

Oleksandr Smirnov linter: add duplicated-account, 1 month ago

package linter

import "olexsmir.xyz/clerk/journal/ast"

// DuplicatedAccount flags account declarations that appear more than once.
type DuplicatedAccount struct{}

func (DuplicatedAccount) ID() RuleID         { return "duplicated-account" }
func (DuplicatedAccount) Severity() Severity { return SeverityWarning }
func (d *DuplicatedAccount) CheckJournal(j *ast.Journal) []Find {
	var finds []Find
	seen := make(map[string]bool)

	for _, entry := range j.Entries {
		ad, ok := entry.(*ast.AccountDirective)
		if !ok {
			continue
		}

		name := ad.Account.String()
		if seen[name] {
			finds = append(finds, Find{
				Code:     d.ID(),
				Severity: d.Severity(),
				Message:  "duplicated account declaration: " + name,
				Span:     ad.Account.Span,
			})
		}
		seen[name] = true
	}
	return finds
}

1	package linter
2
3	import "olexsmir.xyz/clerk/journal/ast"
4
5	// DuplicatedAccount flags account declarations that appear more than once.
6	type DuplicatedAccount struct{}
7
8	func (DuplicatedAccount) ID() RuleID { return "duplicated-account" }
9	func (DuplicatedAccount) Severity() Severity { return SeverityWarning }
10	func (d DuplicatedAccount) CheckJournal(j ast.Journal) []Find {
11	var finds []Find
12	seen := make(map[string]bool)
13
14	for _, entry := range j.Entries {
15	ad, ok := entry.(*ast.AccountDirective)
16	if !ok {
17	continue
18	}
19
20	name := ad.Account.String()
21	if seen[name] {
22	finds = append(finds, Find{
23	Code: d.ID(),
24	Severity: d.Severity(),
25	Message: "duplicated account declaration: " + name,
26	Span: ad.Account.Span,
27	})
28	}
29	seen[name] = true
30	}
31	return finds
32	}