remove endpoint, namespace spec

Author: tonyfromundefined

examples/example.urlspec

@@ -1,13 +1,9 @@
-// Job listing and detail pages
-// This namespace contains all job-related URL specifications
-namespace "jobs";
-
 // Sort order for job listings
 // Used to determine how jobs are displayed
-param sort_order = "recent" | "popular" | "trending";
+param sortOrder = "recent" | "popular" | "trending";
 
 // Status of a job posting
-param job_status = "active" | "closed" | "draft";
+param jobStatus = "active" | "closed" | "draft";
 
 global {
   // Referrer source
@@ -25,7 +21,7 @@ page list = /jobs {
   category?: string;
 
   // Sort order
-  sort: sort_order;
+  sort: sortOrder;
 }
 
 // Job detail page
@@ -38,5 +34,5 @@ page detail = /jobs/:job_id {
   preview?: "true" | "false";
 
   // Job status filter
-  status?: job_status;
+  status?: jobStatus;
 }

packages/builder/src/index.ts

@@ -5,7 +5,6 @@
 import { writeFileSync } from "node:fs";
 import { normalize, resolve } from "node:path";
 import {
-  createEndpoint,
   createGlobalBlock,
   createPageDeclaration,
   createParameterDeclaration,
@@ -27,7 +26,6 @@ import {
 // Re-export AST types
 export type {
   GlobalBlock,
-  NamespaceDeclaration,
   PageDeclaration,
   ParameterDeclaration,
   ParamTypeDeclaration,
@@ -42,9 +40,7 @@ export type {
 } from "@urlspec/language";
 // Re-export AST builder functions for convenience
 export {
-  createEndpoint,
   createGlobalBlock,
-  createNamespace,
   createPageDeclaration,
   createParameterDeclaration,
   createParamTypeDeclaration,
@@ -75,26 +71,10 @@ export interface PageDefinition {
  * URLSpec builder class for programmatic generation of .urlspec files
  */
 export class URLSpec {
-  private namespace?: string;
-  private endpoint?: string;
   private paramTypes: Map<string, ParamType> = new Map();
   private globalParams: ParameterDefinition[] = [];
   private pages: PageDefinition[] = [];
 
-  /**
-   * Set the namespace for the URLSpec
-   */
-  setNamespace(name: string): void {
-    this.namespace = name;
-  }
-
-  /**
-   * Set the endpoint for the URLSpec
-   */
-  setEndpoint(url: string): void {
-    this.endpoint = url;
-  }
-
   /**
    * Add a parameter type definition
    */
@@ -120,10 +100,6 @@ export class URLSpec {
    * Build the URLSpec AST document
    */
   toAST(): URLSpecDocument {
-    if (!this.namespace) {
-      throw new Error("Namespace is required");
-    }
-
     // Build param types
     const paramTypes: ParamTypeDeclaration[] = [];
     for (const [name, type] of this.paramTypes.entries()) {
@@ -149,8 +125,6 @@ export class URLSpec {
     );
 
     return createURLSpecDocument({
-      namespace: this.namespace,
-      endpoint: this.endpoint,
       paramTypes,
       global,
       pages,

packages/builder/test/builder.test.ts

@@ -5,7 +5,6 @@ describe("URLSpec Builder", () => {
   it("should build basic spec", () => {
     const spec = new URLSpec();
 
-    spec.setNamespace("jobs");
     spec.addPage({
       name: "list",
       path: "/jobs",
@@ -14,15 +13,13 @@ describe("URLSpec Builder", () => {
 
     const result = spec.toString();
 
-    expect(result).toContain("namespace jobs;");
     expect(result).toContain("page list = /jobs {");
     expect(result).toContain("category?: string;");
   });
 
   it("should build with param types", () => {
     const spec = new URLSpec();
 
-    spec.setNamespace("jobs");
     spec.addParamType("sortOrder", ["recent", "popular", "trending"]);
     spec.addPage({
       name: "list",
@@ -41,7 +38,6 @@ describe("URLSpec Builder", () => {
   it("should build with global parameters", () => {
     const spec = new URLSpec();
 
-    spec.setNamespace("jobs");
     spec.addGlobalParam({
       name: "utm_source",
       type: "string",
@@ -58,29 +54,24 @@ describe("URLSpec Builder", () => {
     expect(result).toContain("utm_source?: string;");
   });
 
-  it("should build with endpoint", () => {
+  it("should build without namespace and endpoint", () => {
     const spec = new URLSpec();
 
-    spec.setNamespace("jobs");
-    spec.setEndpoint("https://api.example.com");
     spec.addPage({
       name: "list",
       path: "/jobs",
     });
 
     const result = spec.toString();
 
-    expect(result).toContain("namespace jobs;");
-    expect(result).toContain('endpoint "https://api.example.com";');
+    expect(result).not.toContain("namespace");
+    expect(result).not.toContain("endpoint");
     expect(result).toContain("page list = /jobs {");
   });
 
   it("should build complete example", () => {
     const spec = new URLSpec();
 
-    spec.setNamespace("jobs");
-    spec.setEndpoint("https://api.example.com");
-
     spec.addParamType("sortOrder", ["recent", "popular", "trending"]);
     spec.addParamType("jobStatus", ["active", "closed", "draft"]);
 
@@ -111,8 +102,6 @@ describe("URLSpec Builder", () => {
 
     const result = spec.toString();
 
-    expect(result).toContain("namespace jobs;");
-    expect(result).toContain('endpoint "https://api.example.com";');
     expect(result).toContain("param sortOrder");
     expect(result).toContain("param jobStatus");
     expect(result).toContain("global {");
@@ -123,8 +112,6 @@ describe("URLSpec Builder", () => {
   it("should support dynamic page generation", () => {
     const spec = new URLSpec();
 
-    spec.setNamespace("jobs");
-
     const statuses = ["pending", "approved", "rejected"];
     for (const status of statuses) {
       spec.addPage({
@@ -144,7 +131,6 @@ describe("URLSpec Builder", () => {
   describe("Security - Path Traversal Prevention", () => {
     it("should reject paths with .. traversal sequences", async () => {
       const spec = new URLSpec();
-      spec.setNamespace("test");
 
       await expect(spec.writeFile("../../../etc/passwd")).rejects.toThrow(
         "path traversal detected",
@@ -153,7 +139,6 @@ describe("URLSpec Builder", () => {
 
     it("should reject paths to sensitive system directories", async () => {
       const spec = new URLSpec();
-      spec.setNamespace("test");
 
       await expect(spec.writeFile("/etc/hosts")).rejects.toThrow(
         "cannot write to sensitive directory",
@@ -174,7 +159,6 @@ describe("URLSpec Builder", () => {
 
     it("should reject paths outside allowed base directory", async () => {
       const spec = new URLSpec();
-      spec.setNamespace("test");
 
       await expect(
         spec.writeFile("/tmp/test.urlspec", { allowedBaseDir: "/home/user" }),
@@ -183,7 +167,6 @@ describe("URLSpec Builder", () => {
 
     it("should allow valid paths within allowed base directory", async () => {
       const spec = new URLSpec();
-      spec.setNamespace("test");
       spec.addPage({ name: "home", path: "/" });
 
       // This should not throw (but will fail to write without proper setup)
@@ -196,7 +179,6 @@ describe("URLSpec Builder", () => {
 
     it("should normalize paths before validation", async () => {
       const spec = new URLSpec();
-      spec.setNamespace("test");
 
       // These should be rejected even with path obfuscation
       await expect(spec.writeFile("/tmp/../etc/passwd")).rejects.toThrow();