Format files and add tests with CI

[mangalaman93]

No description provided.

[Copilot]

Pull Request Overview

This PR aims to update code formatting, add test coverage, and integrate CI/CD pipelines for both the frontend and backend. Key changes include the addition of CI workflows and lint configurations, new and updated tests for the HTTP echo server, and modifications to the build and run commands.

Reviewed Changes

Copilot reviewed 15 out of 19 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
.github/workflows/ci-tests.yml	Added CI workflow for pull requests with steps for linting, build, and tests.
.golangci.yml	Added configuration for golangci-lint including various linters.
main_test.go	Introduced new test cases to validate GET/POST behaviors and UI asset delivery.
main.go	Merged HTTP method handling and modified the GET query logic; added ReadHeaderTimeout.
Taskfile.yml	Updated tasks to include code formatting and reorganized build/run commands.
ui/*	Various formatting and import style updates across frontend config and components.
.github/workflows/cd-image-push.yml	Updated Docker build/push steps and image naming for CD.
README.md	Updated documentation to reflect the switch to SQLite and new dependency versions.

Files not reviewed (4)

• ui/.prettierignore: Language not supported
• ui/index.html: Language not supported
• ui/package.json: Language not supported
• ui/src/App.css: Language not supported

[Copilot]

Pull Request Overview

This PR introduces enhanced CI/CD workflows, improved logging/error handling in the backend, and additional tests for better quality assurance. Key changes include:

• New GitHub Actions workflows for running tests (ci-tests.yml) and pushing Docker images (cd-image-push.yml) with updated configurations.
• Updates to main.go with standardized logging, revised SQL query ordering, and modifications in HTTP method handling.
• Addition of comprehensive test cases in main_test.go along with various formatting and dependency updates across the project.

Reviewed Changes

Copilot reviewed 16 out of 20 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
.github/workflows/ci-tests.yml	Added CI workflow configuration for running tests
.golangci.yml	Configured golangci linters
main.go	Updated logging, error handling, and SQL query ordering
main_test.go	Added tests for GET/POST routes, UI assets, and concurrent request handling
.github/workflows/cd-image-push.yml	Updated Docker image naming and build configuration
Taskfile.yml	Introduced tasks for building, running, and formatting
ui/eslint.config.js, ui/vite.config.*	Standardized configuration file formatting
ui/src/App.tsx, ui/src/main.tsx, ui/src/components/RequestList.tsx	Updated React app formatting and code style
README.md	Updated documentation to reflect the use of SQLite and new dependencies

Files not reviewed (4)

• ui/.prettierignore: Language not supported
• ui/index.html: Language not supported
• ui/package.json: Language not supported
• ui/src/App.css: Language not supported

Comments suppressed due to low confidence (2)

main.go:114

• The synchronization mechanism (e.g. s.mu.Lock/Unlock) previously used during database inserts in the POST request handling has been removed. Consider reintroducing locking to ensure thread-safe access when writing to the database during concurrent requests.

// Handle POST request

.github/workflows/cd-image-push.yml:13

• [nitpick] Ensure that using '${{ github.repository }}' as the IMAGE_NAME aligns with the intended Docker image naming conventions, as this change may affect how the image is tagged and pushed.

IMAGE_NAME: ${{ github.repository }}

[Copilot]

The check for sql.ErrNoRows after a Query may be redundant because database/sql distinguishes between a query that returns no rows and an actual error. Consider removing this check and handling empty result sets via rows.Next().