Add solution for Challenge 1 by agusu

[agusu]

Summary by CodeRabbit

• New Features
  • Added a simple command-line tool that reads two integers from input, validates them, and prints their sum with clear error messages for invalid input.
  • Introduced a minimal HTTP server with:
    • A public /hello endpoint returning a simple response.
    • A protected /secure endpoint that requires a valid authentication header and returns 401 Unauthorized when missing or invalid.
    • Built-in middleware enforcing authentication for protected routes.
  • Optional local server startup scaffold for easier testing and development.

[coderabbitai]

Walkthrough

Adds a new Go CLI that reads two integers, sums them, and prints the result; and introduces an HTTP server scaffold with an authentication middleware protecting a /secure route while leaving /hello public. Exposes Sum, AuthMiddleware, and SetupServer functions.

Changes

Cohort / File(s)	Summary
CLI Sum Program challenge-1/submissions/agusu/solution-template.go	New Go program: reads two integers from stdin ("%d, %d"), validates input, computes sum via exported Sum(a, b int) int, and prints the result.
HTTP Server with Auth Middleware challenge-5/submissions/agusu/solution-template.go	Adds HTTP server scaffold with AuthMiddleware validating X-Auth-Token == "secret". Routes: /hello (public), /secure (protected). Exposes AuthMiddleware(next http.Handler) http.Handler and SetupServer() http.Handler.

Sequence Diagram(s)

sequenceDiagram autonumber participant User as User (stdin) participant Main as main() participant Sum as Sum(a,b) Note over Main: Parse "%d, %d" from stdin User->>Main: "3, 5" alt parse error Main-->>User: print error and exit else ok Main->>Sum: Sum(3,5) Sum-->>Main: 8 Main-->>User: print "8" end 

Loading

sequenceDiagram autonumber participant Client as Client participant Server as SetupServer() participant MW as AuthMiddleware participant Handler as secureHandler Client->>Server: GET /secure (X-Auth-Token) Server->>MW: Wrap secureHandler MW->>MW: Check header == "secret" alt Unauthorized MW-->>Client: 401 Unauthorized else Authorized MW->>Handler: Serve request Handler-->>Client: 200 OK (secure response) end Client->>Server: GET /hello Server-->>Client: 200 OK (hello) 

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A rabbit taps keys with a cheerful hum,
Adds headers, guards routes, and a tidy Sum.
“Hello,” it squeaks, “secure’s down the hall—
bring the secret, or face a 401 wall.”
Ears up, code done, I thump—ship it to all! 🐇✨

Pre-merge checks

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 71.43% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title correctly identifies the addition of the Challenge 1 solution by the author but omits the inclusion of Challenge 5 changes in the same pull request, yet it still refers to a real part of the changeset. Because it accurately references a legitimate subset of updates, it meets the criteria for a partially related title.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (2)

challenge-1/submissions/agusu/solution-template.go (1)

10-14: Consider exiting with a non-zero status code on error.

The error is printed but the program returns without signaling failure to the shell. This could mask issues in automated testing.

Apply this diff to exit with a non-zero status code:

	_, err := fmt.Scanf("%d, %d", &a, &b)	if err != nil {	fmt.Println("Error reading input:", err) -return +os.Exit(1)	}

Note: You'll need to import the os package:

 import (	"fmt" +"os" )

challenge-5/submissions/agusu/solution-template.go (1)

13-22: Remove unnecessary parentheses and improve error message.

The middleware logic is sound, but has two minor issues:

• Line 16: Unnecessary parentheses around the condition
• Line 17: Empty error message reduces clarity for API consumers

Apply this diff to address these issues:

 func AuthMiddleware(next http.Handler) http.Handler {	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { token := r.Header.Get("X-Auth-Token") - if (token != validToken) { - http.Error(w, "", http.StatusUnauthorized) + if token != validToken { + http.Error(w, "Unauthorized", http.StatusUnauthorized) return } next.ServeHTTP(w, r)	}) }

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 061108a and 1896a45.

📒 Files selected for processing (2)

• challenge-1/submissions/agusu/solution-template.go (1 hunks)
• challenge-5/submissions/agusu/solution-template.go (1 hunks)

🔇 Additional comments (2)

challenge-1/submissions/agusu/solution-template.go (1)

21-24: LGTM!

The Sum function is correctly implemented and properly exported for external use.

challenge-5/submissions/agusu/solution-template.go (1)

34-47: LGTM!

The server setup correctly demonstrates:

• Proper mux configuration
• Public vs. protected routes
• Middleware wrapping pattern