fix(web): mw.CORS rejects wildcard; fix README fences; align to v1.1.2

Coordinated framework release. Documentation-only code changes plus the shared
version bump (folds the never-tagged 1.0.1 contracts pin into 1.1.0).

- mw.Recover() -> mw.Recover(logger): web/mw/doc.go, web/server/doc.go,
  web/server/server.go.
- health.NewHandler(...) -> ....ServeHTTP (chi Get takes HandlerFunc): web/server/doc.go,
  web/doc.go, web/health/doc.go.
- go.mod: contracts, core -> v1.1.0 (via go get + go mod tidy).

Verified by compiling the example patterns against the real API. Badge -> v1.1.0.

Reviewed-on: #1
Co-authored-by: Rene Nochebuena Guerrero <rene@nochebuena.dev>
Co-committed-by: Rene Nochebuena Guerrero <rene@nochebuena.dev>

CHANGELOG.md

@@ -6,13 +6,48 @@ This module adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html
 
 ---
 
-## [1.0.1] — 2026-05-28
+## [1.1.2] — 2026-08-08
+
+Patch — CORS wildcard hardening plus documentation fixes.
 
 ### Changed
 
-- Bumped `code.nochebuena.dev/einherjar/contracts` dependency from v1.0.0 to v1.1.0.
-  No code changes — MVS selects v1.1.0 automatically when a consumer (e.g. `auth`)
-  requires the new `SecurityBag` API. This pin makes the minimum explicit.
+- **`mw.CORS` now rejects `"*"` (panics at construction)** instead of silently no-op'ing it.
+  `"*"` matched nothing (exact-match only), so a service passing it ran with CORS effectively
+  off — a silent trap. Fail loud at boot; use `mw.CORSAllowAll()` (development) or list explicit origins.
+- Bumped `contracts`, `core` to v1.1.2.
+
+### Fixed
+
+- README Go examples now compile: `mw.Recover(logger)`, `health.NewHandler(...).ServeHTTP`, and the
+  `mw.CORS` example no longer passes `"*"`. Corrected the `CORSAllowAll` description.
+
+## [1.1.1] — 2026-08-07
+
+Patch — coordinated framework version alignment.
+
+### Changed
+
+- Bumped `contracts` and `core` to v1.1.1 (framework version alignment). No code or API changes.
+
+## [1.1.0] — 2026-08-07
+
+Coordinated framework release. Documentation fixes plus the framework version bump
+(which finally makes the previously-drafted `contracts` v1.1.0 pin real).
+
+### Fixed
+
+- **Package doc examples didn't compile.** Verified by compiling the example patterns
+  against the real API:
+  - `mw.Recover()` -> `mw.Recover(logger)` — the recover middleware takes a `logging.Logger`
+    (`server`, `mw` package docs and `server.go`).
+  - `health.NewHandler(logger, …)` -> `health.NewHandler(logger, …).ServeHTTP` — the handler
+    returns `http.Handler`, but chi's `Get` takes `http.HandlerFunc`; same for
+    `NewHandlerWithConfig` (`server`, `web`, `health` package docs).
+
+### Changed
+
+- Bumped `contracts` and `core` to v1.1.0 (framework version alignment).
 
 ---
 
@@ -50,7 +85,7 @@ This module adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html
   request logging: method, path, status, latency; uses `StatusRecorder` to capture code
 - `CORS(origins []string) func(http.Handler) http.Handler` — sets
   `Access-Control-Allow-Origin` for listed origins; supports preflight (`OPTIONS`)
-- `CORSAllowAll() func(http.Handler) http.Handler` — shorthand for `CORS([]string{"*"})`
+- `CORSAllowAll() func(http.Handler) http.Handler` — allows any origin by reflecting the request `Origin` (no `Access-Control-Allow-Credentials`); development only
 - `RateLimiterStore` interface — `Allow(ctx context.Context, key string) (bool, error)`;
   pluggable backend; `error` return allows infrastructure failures to surface; fail-open
   contract: non-nil error allows the request

README.md

@@ -1,6 +1,6 @@
 # einherjar/web
 
-[![version](https://img.shields.io/badge/version-v1.0.0-5C4EE5?style=flat-square)](https://code.nochebuena.dev/einherjar/web)
+[![version](https://img.shields.io/badge/version-v1.1.2-5C4EE5?style=flat-square)](https://code.nochebuena.dev/einherjar/web)
 [![license](https://img.shields.io/badge/license-AGPL--3.0-22863A?style=flat-square)](LICENSE)
 [![go](https://img.shields.io/badge/Go-1.26+-00ADD8?style=flat-square&logo=go&logoColor=white)](https://go.dev)
 
@@ -46,7 +46,7 @@ logger := logz.New(logz.Config{JSON: true, StaticArgs: []any{"service", "api"}})
 srv := web.New(logger)
 // Pre-wired stack: Recover → RequestID (UUID v7/v4) → RequestLogger → [CORS]
 
-srv.Get("/health", health.NewHandler(logger, db, cache))
+srv.Get("/health", health.NewHandler(logger, db, cache).ServeHTTP)
 
 lc := launcher.New(logger)
 lc.Append(srv)
@@ -76,7 +76,7 @@ Environment variables for `web.New`:
 | `EINHERJAR_SERVER_WRITE_TIMEOUT` | `10s` | HTTP write timeout |
 | `EINHERJAR_SERVER_IDLE_TIMEOUT` | `120s` | Keep-alive idle timeout |
 | `EINHERJAR_SERVER_SHUTDOWN_TIMEOUT` | `10s` | Graceful shutdown budget |
-| `EINHERJAR_SERVER_CORS_ORIGINS` | _(empty — CORS off)_ | Comma-separated allowed origins |
+| `EINHERJAR_SERVER_CORS_ORIGINS` | _(empty — CORS off)_ | Comma-separated allowed origins (`*` is rejected — use `mw.CORSAllowAll()` in code for allow-all) |
 
 ### Tier 2 — Full control (`server.New`)
 
@@ -90,9 +90,9 @@ import (
 
 srv := server.New(logger, server.Config{Port: 9090},
     server.WithMiddleware(
-        mw.Recover(),
+        mw.Recover(logger),
         mw.RequestID(myIDGenerator),
-        mw.CORS([]string{"*"}),
+        mw.CORS([]string{"https://example.com"}),
         mw.RequestLogger(logger),
         myOwnMiddleware,
     ),
@@ -177,7 +177,7 @@ values are mapped to their canonical HTTP status codes (full 16-code table below
 import "code.nochebuena.dev/einherjar/web/health"
 
 // db and cache implement observability.Checkable
-srv.Get("/health", health.NewHandler(logger, db, cache))
+srv.Get("/health", health.NewHandler(logger, db, cache).ServeHTTP)
 
 // Response shape:
 // {"status":"UP","components":{"db":{"status":"UP","latency":"1.2ms"}}}

doc.go

@@ -18,7 +18,7 @@
 //	lc := launcher.New(logger)
 //
 //	srv := web.New(logger)
-//	srv.Get("/health", health.NewHandler(logger, db, cache))
+//	srv.Get("/health", health.NewHandler(logger, db, cache).ServeHTTP)
 //
 //	lc.Append(srv)
 //	lc.BeforeStart(func() error {

go.mod

@@ -3,8 +3,8 @@ module code.nochebuena.dev/einherjar/web
 go 1.26
 
 require (
-	code.nochebuena.dev/einherjar/contracts v1.0.0
-	code.nochebuena.dev/einherjar/core v1.0.0
+	code.nochebuena.dev/einherjar/contracts v1.1.2
+	code.nochebuena.dev/einherjar/core v1.1.2
 	github.com/go-chi/chi/v5 v5.2.1
 	github.com/google/uuid v1.6.0
 	golang.org/x/time v0.11.0

go.sum

@@ -1,7 +1,7 @@
-code.nochebuena.dev/einherjar/contracts v1.0.0 h1:hRudEtOIqU7vwedYLsCh8+9q5dCnKb61qX+zibqImRU=
-code.nochebuena.dev/einherjar/contracts v1.0.0/go.mod h1:ccltUtrFb5+MEJdkx2VVEUL+xC5pupVlVVsMM8AlCWI=
-code.nochebuena.dev/einherjar/core v1.0.0 h1:AueZgfjp3+rQmDKOxmJQ945TTh+sqC1l/xJdTOdbr9w=
-code.nochebuena.dev/einherjar/core v1.0.0/go.mod h1:0IywfRnJXX9xXQO6iPVaq2QDlXbbpXrB8A4T7gO8nE4=
+code.nochebuena.dev/einherjar/contracts v1.1.2 h1:LNAFCKQjpNjkCyMid2AgkhDPPzOx8dvpRqqZ7F3zpo0=
+code.nochebuena.dev/einherjar/contracts v1.1.2/go.mod h1:ccltUtrFb5+MEJdkx2VVEUL+xC5pupVlVVsMM8AlCWI=
+code.nochebuena.dev/einherjar/core v1.1.2 h1:iYU2fIWtnyOYFbTMD92NaLJMye4XgZHXWhEjB3iS7lo=
+code.nochebuena.dev/einherjar/core v1.1.2/go.mod h1:Y7qZ9nri9Ydey3+40yz75KErOvOUjRSoffdykXbncsw=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/gabriel-vasile/mimetype v1.4.12 h1:e9hWvmLYvtp846tLHam2o++qitpguFiYCKbn0w9jyqw=

health/doc.go

@@ -12,12 +12,12 @@
 //
 // # Example
 //
-//	srv.Get("/health", health.NewHandler(logger, db, cache, storage))
+//	srv.Get("/health", health.NewHandler(logger, db, cache, storage).ServeHTTP)
 //
 // Or with custom timeout:
 //
 //	srv.Get("/health", health.NewHandlerWithConfig(logger,
 //	    health.Config{CheckTimeout: 3 * time.Second},
 //	    db, cache,
-//	))
+//	).ServeHTTP)
 package health

mw/cors.go

@@ -11,6 +11,15 @@ const (
 // Returns 204 No Content for OPTIONS preflight requests.
 // Pass the outermost origins first; an empty slice is a no-op.
 func CORS(origins []string) func(http.Handler) http.Handler {
+	// "*" is a silent no-op here (exact-match only) — reject it loudly at
+	// construction so a misconfigured service fails to boot instead of quietly
+	// blocking every browser. For allow-all, call CORSAllowAll (development only).
+	for _, o := range origins {
+		if o == "*" {
+			panic(`mw.CORS: "*" is not a valid origin — list explicit origins, or use CORSAllowAll() for allow-all`)
+		}
+	}
+
 	originSet := make(map[string]struct{}, len(origins))
 	for _, o := range origins {
 		originSet[o] = struct{}{}

mw/doc.go

@@ -6,7 +6,7 @@
 // # Recommended middleware order (outermost first)
 //
 //	server.WithMiddleware(
-//	    mw.Recover(),
+//	    mw.Recover(logger),
 //	    mw.RequestID(uuid.NewString),
 //	    mw.RequestLogger(logger),
 //	    mw.CORS([]string{"https://example.com"}),

server/doc.go

@@ -12,14 +12,14 @@
 //
 //	srv := server.New(logger, server.Config{Port: 8080},
 //	    server.WithMiddleware(
-//	        mw.Recover(),
+//	        mw.Recover(logger),
 //	        mw.RequestID(myIDGenerator),
 //	        mw.RequestLogger(logger),
 //	        mw.CORS([]string{"https://example.com"}),
 //	    ),
 //	)
 //
-//	srv.Get("/health", health.NewHandler(logger, db))
+//	srv.Get("/health", health.NewHandler(logger, db).ServeHTTP)
 //
 //	lc := launcher.New(logger)
 //	lc.Append(srv)

server/server.go

@@ -7,7 +7,7 @@
 //
 //	srv := server.New(logger, server.Config{Port: 8080},
 //	    server.WithMiddleware(
-//	        mw.Recover(),
+//	        mw.Recover(logger),
 //	        mw.RequestID(uuid.NewString),
 //	        mw.RequestLogger(logger),
 //	    ),