Fix linting errors. Add pre-commit

Author: Ch00k

.golangci.yml

@@ -0,0 +1,17 @@
+version: "2"
+linters:
+  settings:
+    staticcheck:
+      checks:
+        - all
+  exclusions:
+    paths:
+      - ui/components
+      - ui/utils/templui.go
+formatters:
+  enable:
+    - golines
+  settings:
+    golines:
+      max-len: 120
+      tab-len: 4

.pre-commit-config.yaml

@@ -0,0 +1,19 @@
+repos:
+  - repo: https://github.com/tekwizely/pre-commit-golang
+    rev: v1.0.0-rc.2
+    hooks:
+      - id: go-mod-tidy
+
+  - repo: https://github.com/golangci/golangci-lint
+    rev: v2.5.0
+    hooks:
+      - id: golangci-lint-full
+
+  - repo: local
+    hooks:
+      - id: prettier
+        name: prettier
+        entry: make prettier
+        verbose: true
+        language: system
+        types: [css, javascript, html]

Makefile

@@ -8,10 +8,12 @@ CR_WEB_ASSETS_FILENAME ?= frontend-assets.tar.gz
 CR_BUILD_ARTIFACTS_DIR ?= dist
 
 
-lint:
-	golangci-lint run --fix
+prettier:
 	prettier --write frontend/
 
+lint: prettier
+	golangci-lint run --fix
+
 test:
 	gotestsum --format testname ./...
 

db.go

@@ -51,7 +51,7 @@ func initDB() error {
 
 	// Create claude-review directory
 	dbDir := filepath.Join(dataHome, "claude-review")
-	if err := os.MkdirAll(dbDir, 0755); err != nil {
+	if err := os.MkdirAll(dbDir, 0o755); err != nil {
 		return fmt.Errorf("failed to create database directory: %w", err)
 	}
 
@@ -178,31 +178,6 @@ func getComments(projectDir, filePath string, resolved bool) ([]Comment, error)
 	return comments, nil
 }
 
-func getAllCommentsForFile(projectDir, filePath string) ([]Comment, error) {
-	query := `
-		SELECT id, project_directory, file_path, line_start, line_end, selected_text, comment_text, resolved, created_at
-		FROM comments
-		WHERE project_directory = ? AND file_path = ?
-		ORDER BY resolved ASC, line_start ASC`
-	logQuery(query, projectDir, filePath)
-	rows, err := db.Query(query, projectDir, filePath)
-	if err != nil {
-		return nil, err
-	}
-	defer func() { _ = rows.Close() }()
-
-	var comments []Comment
-	for rows.Next() {
-		var c Comment
-		if err := rows.Scan(&c.ID, &c.ProjectDirectory, &c.FilePath, &c.LineStart, &c.LineEnd, &c.SelectedText, &c.CommentText, &c.Resolved, &c.CreatedAt); err != nil {
-			return nil, err
-		}
-		comments = append(comments, c)
-	}
-
-	return comments, nil
-}
-
 func updateComment(commentID, commentText string) error {
 	query := `
 		UPDATE comments

frontend/static/viewer.js

@@ -130,7 +130,7 @@
         commentButton.style.left = x + 'px';
         // Offset by button height to align bottom of button with top of selection
         const buttonHeight = 40; // 32px icon + 2*4px padding
-        commentButton.style.top = (y - buttonHeight) + 'px';
+        commentButton.style.top = y - buttonHeight + 'px';
     }
 
     function hideCommentButton() {

markdown.go

@@ -129,7 +129,12 @@ func (r *CodeBlockRenderer) RegisterFuncs(reg renderer.NodeRendererFuncRegistere
 	reg.Register(ast.KindCodeBlock, r.renderCodeBlock)
 }
 
-func (r *CodeBlockRenderer) renderCodeBlock(w util.BufWriter, source []byte, node ast.Node, entering bool) (ast.WalkStatus, error) {
+func (r *CodeBlockRenderer) renderCodeBlock(
+	w util.BufWriter,
+	source []byte,
+	node ast.Node,
+	entering bool,
+) (ast.WalkStatus, error) {
 	if entering {
 		// Write <pre> with data-line attributes
 		_, _ = w.WriteString("<pre")