work by vigo · Pull Request #1 · bilustek/avokado

vigo · 2026-03-25T19:46:19Z

implement avokado and errors package
add badges

Copilot

Pull request overview

Introduces the initial avokado Fiber server builder and an avoerror package for typed API errors, along with repository/CI scaffolding (lint/test workflows, pre-commit, badges, and community files).

Changes:

Add avokado server constructor with functional options and a built-in /healthz route.
Add avoerror package with fluent error builder + tests.
Add CI (lint/tests + coverage upload), repo metadata/config (golangci, pre-commit, badges, CODEOWNERS, license, CoC).

Reviewed changes

Copilot reviewed 13 out of 15 changed files in this pull request and generated 12 comments.

Show a summary per file

File	Description
go.mod	Defines module + Go version + dependencies.
go.sum	Adds dependency checksums.
avokado.go	Implements server builder/options, listen/shutdown, and health endpoint.
avoerror/avoerror.go	Adds typed API error type + fluent helpers.
avoerror/avoerror_test.go	Adds tests for error behavior and constants.
README.md	Adds badges and basic project description.
.github/workflows/go-test.yml	Adds Go build/test + coverage upload workflow.
.github/workflows/go-lint.yml	Adds golangci-lint workflow.
.golangci.yml	Configures linters/formatters.
.pre-commit-config.yaml	Adds pre-commit hooks for Go.
.gitignore	Adds Go-specific ignore rules.
LICENSE	Adds MIT license.
CODE_OF_CONDUCT.md	Adds Contributor Covenant CoC.
.github/CODEOWNERS	Adds default code owner.
.bumpversion.toml	Adds version bump configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-25T19:50:01Z

+	config *config
+}
+
+// ListenAndServe starts fiber server with grafeful shutdown.


Typo in comment: "grafeful" should be "graceful".

Suggested change

// ListenAndServe starts fiber server with grafeful shutdown.

// ListenAndServe starts fiber server with graceful shutdown.

Copilot · 2026-03-25T19:50:02Z

+		if d < 0 {
+			return errors.New("fix this")
+			// return fmt.Errorf(
+			// 	"[webhookserver.WithIdleTimeout] error: [%w, '%s' received, must > 0]",
+			// 	cerrors.ErrInvalid, d,
+			// )
+		}


This option currently returns errors.New("fix this") for invalid input and includes commented-out code. Replace the placeholder with a specific, stable error (ideally a package-level sentinel) and remove the dead commented block so callers get actionable feedback.

Copilot · 2026-03-25T19:50:02Z

+		if d < 0 {
+			return errors.New("fix this")
+			// return fmt.Errorf(
+			// 	"[webhookserver.WithReadTimeout] error: [%w, '%s' received, must > 0]",
+			// 	cerrors.ErrInvalid, d,
+			// )
+		}


This option currently returns errors.New("fix this") for invalid input and includes commented-out code. Replace the placeholder with a specific, stable error (ideally a package-level sentinel) and remove the dead commented block so callers get actionable feedback.

Copilot · 2026-03-25T19:50:02Z

+
+	var target *avoerror.Error
+	if !errors.As(wrapped, &target) {
+		t.Fatal("errors.As should find *avokadoerror.Error in chain")


The failure message references *avokadoerror.Error, but the type under test is *avoerror.Error. This makes the test output misleading when it fails.

Suggested change

t.Fatal("errors.As should find *avokadoerror.Error in chain")

t.Fatal("errors.As should find *avoerror.Error in chain")

Copilot · 2026-03-25T19:50:02Z

+![Go Build Status](https://github.com/bilustek/avokado/actions/workflows/go-lint.yml/badge.svg)
+![GolangCI-Lint Status](https://github.com/bilustek/avokado/actions/workflows/go-test.yml/badge.svg)


The two workflow badges appear swapped: the "Go Build Status" badge points to go-lint.yml, and the "GolangCI-Lint Status" badge points to go-test.yml. Update the labels or URLs so each badge matches the workflow it represents.

Suggested change

![Go Build Status](https://github.com/bilustek/avokado/actions/workflows/go-lint.yml/badge.svg)

![GolangCI-Lint Status](https://github.com/bilustek/avokado/actions/workflows/go-test.yml/badge.svg)

![Go Build Status](https://github.com/bilustek/avokado/actions/workflows/go-test.yml/badge.svg)

![GolangCI-Lint Status](https://github.com/bilustek/avokado/actions/workflows/go-lint.yml/badge.svg)

Copilot · 2026-03-25T19:50:03Z

+	if cfg.serverAPIprefix == "" {
+		cfg.serverAPIprefix = "/api/v1"
+	}


serverAPIprefix is configurable (option + default) but never used when registering routes or grouping handlers, so it currently has no effect and may confuse API consumers. Either apply it (e.g., mount a group under the prefix) or remove it until it's supported.

Copilot · 2026-03-25T19:50:03Z

+
+	select {
+	case <-ctx.Done():
+		return s.App.ShutdownWithContext(context.Background())


ListenAndServe triggers shutdown using context.Background(), which drops cancellation/deadline information from the incoming context. Prefer deriving a shutdown context (often with a timeout) from ctx so shutdown can be bounded and responds to cancellation appropriately.

Suggested change

return s.App.ShutdownWithContext(context.Background())

shutdownTimeout := s.config.idleTimeout

if shutdownTimeout <= 0 {

shutdownTimeout = defaultIdleTimeout

}

shutdownCtx, cancel := context.WithTimeout(ctx, shutdownTimeout)

defer cancel()

return s.App.ShutdownWithContext(shutdownCtx)

Copilot · 2026-03-25T19:50:04Z

+		if d < 0 {
+			return errors.New("fix this")
+			// return fmt.Errorf(
+			// 	"[webhookserver.WithReadTimeout] error: [%w, '%s' received, must > 0]",
+			// 	cerrors.ErrInvalid, d,
+			// )
+		}


This option currently returns errors.New("fix this") for invalid input and includes commented-out code. Replace the placeholder with a specific, stable error (ideally a package-level sentinel) and remove the dead commented block so callers get actionable feedback.

Copilot · 2026-03-25T19:50:04Z

+
+# avokado
+
+Quick rest api server genarator for nerds! Includes:


Spelling/grammar: "genarator" → "generator".

Suggested change

Quick rest api server genarator for nerds! Includes:

Quick rest api server generator for nerds! Includes:

Copilot · 2026-03-25T19:50:04Z

+	}
+
+	fiberCfg := *cfg.fiberConfig
+	fiberCfg.AppName = cfg.serverName


idleTimeout/readTimeout/writeTimeout are configured (and have options/defaults) but never applied to the Fiber config before creating the app, so the options currently have no effect. Wire these values into fiberCfg (or the appropriate listen/server config fields) prior to fiber.New.

Suggested change

fiberCfg.AppName = cfg.serverName

fiberCfg.AppName = cfg.serverName

fiberCfg.IdleTimeout = cfg.idleTimeout

fiberCfg.ReadTimeout = cfg.readTimeout

fiberCfg.WriteTimeout = cfg.writeTimeout

sentry · 2026-03-25T20:05:58Z

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

vigo added 2 commits March 25, 2026 22:39

implement avokado and errors package

f165cf8

add badges

dddc209

Copilot AI review requested due to automatic review settings March 25, 2026 19:46

Copilot started reviewing on behalf of vigo March 25, 2026 19:46 View session

Copilot AI reviewed Mar 25, 2026

View reviewed changes

fix errors

501d861

vigo added 23 commits March 25, 2026 23:15

add avologger

8e0dc70

update README

d68e221

add IsSentryEnabled

80eeccf

fix typos

6934bb9

fix context

5cfb99d

add avokado tests

f5f7607

add graceful shutdown test

3b6c081

add avokado version

de9a7f7

add avomiddleware - Sentry

99c37ba

add avoresponse

124f7fb

fix http status codes

c8a3faf

fix base http handler -> exportable

49c54cf

improve tests

ad4462f

wip

343d8f6

wip

f9dea13

improve custom validation messages

d60407a

add avodb package

3ab31eb

update README

4475045

improve tests

414ab9c

rename to avokadodb

43bb5ac

rename to avokadologger

c0d4725

rename

4ea5f75

rename

2d60610

vigo added 22 commits March 26, 2026 22:57

rename

0961013

update

1fb6518

add migrate

21f98fe

fix github action

1cc9f09

add squawk pre-commit hook

3ba31f3

add auth_user

39d479b

add refresh_token

b4b42fa

add refresh_token

b801993

wip

1114593

add other tables

e622161

up

b56ef36

update README

d000adc

improve avokadoerror

7173a6b

fix migration version

ef62e79

improve migrate

207244e

fix test

74ba176

add makemigration and string utils

3713d2d

fix avokado error usage and disable revives line limit

c7e5ea7

add management commands info

2fa2dc3

update readme

f5988ad

update readme

76dc4af

implement avokadonotifier package (#2)

0a4d21a

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

work#1

work#1
vigo wants to merge 48 commits into
mainfrom
work

vigo commented Mar 25, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

sentry Bot commented Mar 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	// ListenAndServe starts fiber server with grafeful shutdown.
	// ListenAndServe starts fiber server with graceful shutdown.

	t.Fatal("errors.As should find *avokadoerror.Error in chain")
	t.Fatal("errors.As should find *avoerror.Error in chain")

		![Go Build Status](https://github.com/bilustek/avokado/actions/workflows/go-lint.yml/badge.svg)
		![GolangCI-Lint Status](https://github.com/bilustek/avokado/actions/workflows/go-test.yml/badge.svg)

-		return s.App.ShutdownWithContext(context.Background())
+		shutdownTimeout := s.config.idleTimeout
+		if shutdownTimeout <= 0 {
+			shutdownTimeout = defaultIdleTimeout
+		}
+		shutdownCtx, cancel := context.WithTimeout(ctx, shutdownTimeout)
+		defer cancel()
+		return s.App.ShutdownWithContext(shutdownCtx)


		# avokado

		Quick rest api server genarator for nerds! Includes:

	Quick rest api server genarator for nerds! Includes:
	Quick rest api server generator for nerds! Includes:

-	fiberCfg.AppName = cfg.serverName
+	fiberCfg.AppName = cfg.serverName
+	fiberCfg.IdleTimeout = cfg.idleTimeout
+	fiberCfg.ReadTimeout = cfg.readTimeout
+	fiberCfg.WriteTimeout = cfg.writeTimeout

Conversation

vigo commented Mar 25, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

sentry Bot commented Mar 25, 2026

Welcome to Codecov 🎉

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants