beast/v2

[sukhman-sukh]

No description provided.

[Copilot]

Pull request overview

This PR introduces significant architectural changes to migrate Beast from SQLite to PostgreSQL, implements remote server deployment capabilities, adds OTP-based email verification, and enhances the leaderboard system with frozen scores and time-series data visualization.

Key Changes:

• Migration from SQLite to PostgreSQL with comprehensive database backup/restore functionality
• Remote server deployment support via SSH with load balancing across multiple servers
• OTP-based email verification for user registration and password recovery
• Enhanced leaderboard with frozen scoring, pagination, and time-series graph data for top users
• Challenge hints system with point-based cost and attempt limits

Reviewed changes

Copilot reviewed 72 out of 2151 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
core/database/database.go	Migrated from SQLite to PostgreSQL with backup/restore capabilities
pkg/remoteManager/*.go	Added remote server management for distributed challenge deployment
api/otp.go	Implemented OTP-based email verification system
core/database/hints.go	Added hints system with point-based unlocking
api/info.go	Enhanced leaderboard with frozen scores and time-series data
core/database/challenges.go	Added challenge attempt limits, prerequisites, and solve tracking
utils/prompt.go	Added interactive CLI prompts for configuration
cmd/beast/init.go	Comprehensive initialization with database and config setup
go.mod	Updated Go version to 1.22.0 and dependencies

Comments suppressed due to low confidence (1)

api/submit.go:1

• String concatenation should use fmt.Sprintf for better readability and maintainability, especially with multiple variables.

package api

---

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

[Copilot]

The defer statement for file.Close() is positioned after the error check, meaning the file will not be closed if an error occurs. Move the defer statement immediately after the file assignment to ensure proper resource cleanup even in error cases.

[Copilot]

The defer statement is inside a loop, which means client.Close() will only be called when Init() returns, not after each iteration. This will leave SSH connections open until the function exits. Move client.Close() outside the defer to close it immediately after use within the loop iteration.

Suggested change

	defer client.Close()
	ServerQueue.Push(server)
	RunCommandOnServer(server, "mkdir -p $HOME/.beast/staging/")
	ServerQueue.Push(server)
	RunCommandOnServer(server, "mkdir -p $HOME/.beast/staging/")
	client.Close()

[Copilot]

The error handling returns a default time bucket (Between1MonthAnd1Year) on parsing failure, which could mask configuration errors. Consider returning a specific error bucket or logging more context about why parsing failed.

[Copilot]

InsecureSkipVerify is set to true, which disables TLS certificate verification and makes the connection vulnerable to man-in-the-middle attacks. This should be set to false in production or made configurable.

Suggested change

	InsecureSkipVerify: true, // Set true only if SMTP server uses self-signed certs
	InsecureSkipVerify: false, // Do not skip verification; secure by default

[Copilot]

Corrected spelling of 'Faliure' to 'Failure'.

Suggested change

	msg := fmt.Sprintf("SERVER HEALTH CHECK Faliure: %s : %s", server.Host, err)
	msg := fmt.Sprintf("SERVER HEALTH CHECK Failure: %s : %s", server.Host, err)

[Copilot]

Using fmt.Sprint with a single string argument is unnecessary. The string can be used directly without formatting.

Suggested change

	Message: fmt.Sprint("challenge name cannot be empty"),
	Message: "challenge name cannot be empty",