docs(web): document CORS wildcard rejection + env-gated convention; align to v1.1.3

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,62 @@ This module adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html
 
 ---
 
-## [1.0.1] — 2026-05-28
+## [1.1.3] — 2026-08-08
+
+Patch — CORS documentation discoverability.
+
+### Fixed
+
+- `mw.CORS` and `CORSAllowAll` doc comments now document the `"*"` rejection (panic) and the
+  env-gated CORS convention (`local -> CORSAllowAll`, else `mw.CORS(origins)`), so `search_symbols`
+  surfaces it — previously the convention lived only in code comments and the wire example.
 
 ### Changed
 
-- Bumped `code.nochebuena.dev/einherjar/contracts` dependency from v1.0.0 to v1.1.0.
+- Bumped `contracts`, `core` to v1.1.3.
-  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.
+## [1.1.2] — 2026-08-08
+
+Patch — CORS wildcard hardening plus documentation fixes.
+
+### Changed
+
+- **`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 +99,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.3-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/contracts v1.1.3
-	code.nochebuena.dev/einherjar/core v1.0.0
+	code.nochebuena.dev/einherjar/core v1.1.3
 	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.1.3 h1:rBtQUVCeqaIKMcG1+R0ndHM/4cRm3XTFnVNzFTbf1QU=
-code.nochebuena.dev/einherjar/contracts v1.0.0/go.mod h1:ccltUtrFb5+MEJdkx2VVEUL+xC5pupVlVVsMM8AlCWI=
+code.nochebuena.dev/einherjar/contracts v1.1.3/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.1.3 h1:MjWUA/hJ5IOosACh0in9xzFZ7jc1TTO0YEHh1xQVuOg=
-code.nochebuena.dev/einherjar/core v1.0.0/go.mod h1:0IywfRnJXX9xXQO6iPVaq2QDlXbbpXrB8A4T7gO8nE4=
+code.nochebuena.dev/einherjar/core v1.1.3/go.mod h1:xkI2uQ4S0FQHbFiawTUFgJdG5jbivcZpGFddHxQe8Q0=
 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

@@ -7,10 +7,29 @@ const (
 	allowedHeaders = "Content-Type, Authorization, X-Request-ID"
 )
 
-// CORS sets cross-origin resource sharing headers for the provided origins.
+// CORS sets cross-origin resource sharing headers for the provided origins
-// Returns 204 No Content for OPTIONS preflight requests.
+// (exact match; an empty slice is a no-op). Returns 204 No Content for OPTIONS
-// Pass the outermost origins first; an empty slice is a no-op.
+// preflight requests.
+//
+// It panics on "*": a wildcard matches no real Origin here, so passing it would
+// silently disable CORS. For allow-all use [CORSAllowAll] (development only). The
+// recommended wiring gates CORS by environment:
+//
+//	var corsMW func(http.Handler) http.Handler
+//	if strings.EqualFold(cfg.AppEnv, "local") {
+//	    corsMW = mw.CORSAllowAll()            // dev: any origin
+//	} else {
+//	    corsMW = mw.CORS(cfg.AllowedOrigins)  // prod: explicit origins
+//	}
 func CORS(origins []string) func(http.Handler) http.Handler {
+	// "*" would be a silent no-op (exact-match only) — reject it loudly so a
+	// misconfigured service fails to boot instead of quietly blocking browsers.
+	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{}{}
@@ -37,8 +56,12 @@ func CORS(origins []string) func(http.Handler) http.Handler {
 	}
 }
 
-// CORSAllowAll is a convenience wrapper that allows any origin.
+// CORSAllowAll allows any origin by reflecting the request Origin (it does not set
-// Use only in development — never in production.
+// Access-Control-Allow-Credentials). Development only — never in production.
+//
+// Use it for the local branch of the env-gated CORS convention; use [CORS] with
+// explicit origins everywhere else. Because [CORS] panics on "*", CORSAllowAll — not
+// a "*" in the origins list — is the way to allow all.
 func CORSAllowAll() func(http.Handler) http.Handler {
 	return func(next http.Handler) http.Handler {
 		return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {

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),
 //	    ),