@webjsdev/core 0.7.13 → 0.7.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@webjsdev/core",
-  "version": "0.7.13",
+  "version": "0.7.15",
   "type": "module",
   "description": "webjs core runtime - html/css tags, WebComponent base, isomorphic renderers",
   "types": "./index.d.ts",
@@ -75,7 +75,7 @@
     "prepare": "node -e \"require('fs').existsSync('../../scripts/build-framework-dist.js') && require('fs').existsSync('./src/html.js') && require('child_process').execSync('npm run build:dist', {stdio:'inherit'})\""
   },
   "devDependencies": {
-    "esbuild": "^0.24.0"
+    "esbuild": "^0.25.0"
   },
   "publishConfig": {
     "access": "public"

package/src/component.d.ts

@@ -114,7 +114,11 @@ export abstract class WebComponent extends HTMLElement {
   /** Optional render-error boundary inside the component. */
   renderError?(error: Error): TemplateResult | void;
 
-  connectedCallback?(): void;
-  disconnectedCallback?(): void;
-  attributeChangedCallback?(name: string, oldValue: string | null, newValue: string | null): void;
+  // Concrete on the base (component.js implements all three), so a subclass
+  // override can call `super.connectedCallback()` etc. without a
+  // possibly-undefined error (#433). NOT optional, unlike the subclass-only
+  // hooks above (firstUpdated / renderError).
+  connectedCallback(): void;
+  disconnectedCallback(): void;
+  attributeChangedCallback(name: string, oldValue: string | null, newValue: string | null): void;
 }

package/src/nav.d.ts

@@ -1,4 +1,4 @@
 export function notFound(): never;
-export function redirect(url: string, status?: number): never;
+export function redirect(url: string, status?: number | { status?: number }): never;
 export function isNotFound(e: unknown): boolean;
 export function isRedirect(e: unknown): boolean;

package/src/nav.js

@@ -22,15 +22,34 @@ export function notFound() {
 }
 
 /**
+ * Throw a redirect sentinel the SSR pipeline catches and turns into a 3xx.
+ *
+ * The status is OPTIONAL and, when omitted, is chosen by the catching site so
+ * each kind of redirect gets the conventional code:
+ *   - thrown during a GET page/layout render (a gating redirect like an auth
+ *     bounce to `/login`): 302 Found, the conventional GET-to-GET code.
+ *   - thrown from a server action (a POST): 307 Temporary Redirect, which is
+ *     method-preserving so the action's intent survives. (The PRG success
+ *     path uses 303 separately.)
+ * Pass an explicit status to override: `redirect('/x', 308)` (permanent) or the
+ * options form `redirect('/x', { status: 301 })`. An explicit status wins at
+ * every catching site.
+ *
  * @param {string} url
- * @param {number} [status] 307 (temp) by default; pass 308 for permanent
+ * @param {number | { status?: number }} [status] explicit status (number) or
+ *   `{ status }`; omit to let the catching site pick (302 for a GET gate, 307
+ *   for an action).
  * @returns {never}
  */
-export function redirect(url, status = 307) {
+export function redirect(url, status) {
+  const code = typeof status === 'object' && status !== null ? status.status : status;
   const err = new Error(`webjs: redirect(${url})`);
   /** @type any */ (err).__webjs = REDIRECT;
   /** @type any */ (err).url = url;
-  /** @type any */ (err).status = status;
+  // Left undefined when the caller did not specify one, so the catching site
+  // can apply its convention (302 GET gate vs 307 action). A caller-supplied
+  // code is stored verbatim and overrides that convention.
+  /** @type any */ (err).status = typeof code === 'number' ? code : undefined;
   throw err;
 }