@lsst/pik-core 0.6.5 → 0.6.6

package/dist/index.js

@@ -46,6 +46,25 @@ class CommentStyle {
     const ext = extension.replace(/^\./, "").toLowerCase();
     return CommentStyle.styles[ext] ?? CommentStyle.defaultStyle;
   }
+  /**
+   * Get comment style for a file path, handling dotfiles like .env correctly.
+   *
+   * Unlike fromExtension(), this method properly handles:
+   * - `.env` files (where extname returns '')
+   * - `.env.local`, `.env.development`, etc. (where extname returns '.local', '.development')
+   */
+  static fromFilePath(filePath) {
+    const basename = filePath.split(/[/\\]/).pop() ?? "";
+    if (basename === ".env" || basename.startsWith(".env.")) {
+      return CommentStyle.styles["env"] ?? CommentStyle.defaultStyle;
+    }
+    const lastDot = basename.lastIndexOf(".");
+    if (lastDot > 0) {
+      const ext = basename.slice(lastDot + 1).toLowerCase();
+      return CommentStyle.styles[ext] ?? CommentStyle.defaultStyle;
+    }
+    return CommentStyle.defaultStyle;
+  }
   /**
    * Register a custom comment style for an extension
    */
@@ -93,6 +112,12 @@ class Parser {
   static forExtension(extension) {
     return new Parser(CommentStyle.fromExtension(extension));
   }
+  /**
+   * Create a parser for a file path, correctly handling dotfiles like .env
+   */
+  static forFilePath(filePath) {
+    return new Parser(CommentStyle.fromFilePath(filePath));
+  }
   /**
    * Parse content string for pik selectors and options
    */
@@ -213,6 +238,13 @@ class CommentManipulator {
     const { blockOpen, blockClose } = this.commentStyle;
     return trimmed.startsWith(blockOpen) && trimmed.includes(blockClose);
   }
+  /**
+   * Check if a line is a line comment (not a block comment)
+   */
+  isLineComment(line) {
+    const trimmed = line.trimStart();
+    return trimmed.startsWith(this.commentStyle.lineComment);
+  }
   /**
    * Comment out a line if not already commented.
    * Uses the same comment style as the original if block-commented,
@@ -234,30 +266,48 @@ class CommentManipulator {
     return `${indent}${this.commentStyle.lineComment} ${trimmed}`;
   }
   /**
-   * Uncomment a line if commented (handles both line and block comments)
+   * Uncomment a block comment
+   * @throws Error if the line is not a block comment
    */
-  uncommentLine(line) {
+  uncommentBlockComment(line) {
+    if (!this.isBlockCommented(line)) {
+      throw new Error("Expected line to be a block comment");
+    }
     const trimmed = line.trimStart();
     const indent = line.slice(0, line.length - trimmed.length);
-    if (this.commentStyle.hasBlockComments) {
-      const { blockOpen, blockClose } = this.commentStyle;
-      if (trimmed.startsWith(blockOpen)) {
-        const afterOpen = trimmed.slice(blockOpen.length);
-        const closeIndex = afterOpen.indexOf(blockClose);
-        if (closeIndex !== -1) {
-          const content2 = afterOpen.slice(0, closeIndex).trim();
-          const rest = afterOpen.slice(closeIndex + blockClose.length);
-          return `${indent}${content2}${rest}`;
-        }
-      }
-    }
-    if (!trimmed.startsWith(this.commentStyle.lineComment)) {
-      return line;
+    const { blockOpen, blockClose } = this.commentStyle;
+    const afterOpen = trimmed.slice(blockOpen.length);
+    const closeIndex = afterOpen.indexOf(blockClose);
+    const content = afterOpen.slice(0, closeIndex).trim();
+    const rest = afterOpen.slice(closeIndex + blockClose.length);
+    return `${indent}${content}${rest}`;
+  }
+  /**
+   * Uncomment a line comment
+   * @throws Error if the line is not a line comment
+   */
+  uncommentLineComment(line) {
+    if (!this.isLineComment(line)) {
+      throw new Error("Expected line to be a line comment");
     }
+    const trimmed = line.trimStart();
+    const indent = line.slice(0, line.length - trimmed.length);
     const withoutComment = trimmed.slice(this.commentStyle.lineComment.length);
     const content = withoutComment.startsWith(" ") ? withoutComment.slice(1) : withoutComment;
     return `${indent}${content}`;
   }
+  /**
+   * Uncomment a line if commented (handles both line and block comments)
+   */
+  uncommentLine(line) {
+    if (this.isBlockCommented(line)) {
+      return this.uncommentBlockComment(line);
+    }
+    if (this.isLineComment(line)) {
+      return this.uncommentLineComment(line);
+    }
+    return line;
+  }
 }
 class Switcher extends CommentManipulator {
   /**
@@ -279,6 +329,12 @@ class SingleSwitcher extends Switcher {
   static forExtension(extension) {
     return new SingleSwitcher(CommentStyle.fromExtension(extension));
   }
+  /**
+   * Create a switcher for a file path, correctly handling dotfiles like .env
+   */
+  static forFilePath(filePath) {
+    return new SingleSwitcher(CommentStyle.fromFilePath(filePath));
+  }
   /**
    * Switch to a specific option, deactivating all others
    */

package/dist/lib/comment-manipulator.d.ts

@@ -13,12 +13,26 @@ export declare abstract class CommentManipulator {
      * Check if a line uses block comment style
      */
     protected isBlockCommented(line: string): boolean;
+    /**
+     * Check if a line is a line comment (not a block comment)
+     */
+    private isLineComment;
     /**
      * Comment out a line if not already commented.
      * Uses the same comment style as the original if block-commented,
      * otherwise uses line comment style.
      */
     protected commentLine(line: string, useBlockStyle?: boolean): string;
+    /**
+     * Uncomment a block comment
+     * @throws Error if the line is not a block comment
+     */
+    private uncommentBlockComment;
+    /**
+     * Uncomment a line comment
+     * @throws Error if the line is not a line comment
+     */
+    private uncommentLineComment;
     /**
      * Uncomment a line if commented (handles both line and block comments)
      */

package/dist/lib/comment-manipulator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"comment-manipulator.d.ts","sourceRoot":"","sources":["../../src/lib/comment-manipulator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,kBAAkB,CAAC;AAEhD;;GAEG;AACH,8BAAsB,kBAAkB;IAC1B,SAAS,CAAC,QAAQ,CAAC,YAAY,EAAE,YAAY;gBAA1B,YAAY,EAAE,YAAY;IAEzD;;OAEG;IACH,SAAS,CAAC,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAmBhD;;OAEG;IACH,SAAS,CAAC,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAUjD;;;;OAIG;IACH,SAAS,CAAC,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,aAAa,UAAQ,GAAG,MAAM;IAwBlE;;OAEG;IACH,SAAS,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM;CAkC9C"}
+{"version":3,"file":"comment-manipulator.d.ts","sourceRoot":"","sources":["../../src/lib/comment-manipulator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,kBAAkB,CAAC;AAEhD;;GAEG;AACH,8BAAsB,kBAAkB;IAC1B,SAAS,CAAC,QAAQ,CAAC,YAAY,EAAE,YAAY;gBAA1B,YAAY,EAAE,YAAY;IAEzD;;OAEG;IACH,SAAS,CAAC,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAmBhD;;OAEG;IACH,SAAS,CAAC,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAUjD;;OAEG;IACH,OAAO,CAAC,aAAa;IAKrB;;;;OAIG;IACH,SAAS,CAAC,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,aAAa,UAAQ,GAAG,MAAM;IAwBlE;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAoB7B;;;OAGG;IACH,OAAO,CAAC,oBAAoB;IAiB5B;;OAEG;IACH,SAAS,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM;CAa9C"}

package/dist/lib/parser.d.ts

@@ -12,6 +12,10 @@ export declare class Parser {
      * Create a parser for a specific file extension
      */
     static forExtension(extension: string): Parser;
+    /**
+     * Create a parser for a file path, correctly handling dotfiles like .env
+     */
+    static forFilePath(filePath: string): Parser;
     /**
      * Parse content string for pik selectors and options
      */

package/dist/lib/parser.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"parser.d.ts","sourceRoot":"","sources":["../../src/lib/parser.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAU,WAAW,EAAY,MAAM,kBAAkB,CAAC;AACtE,OAAO,EAAE,YAAY,EAAE,MAAM,kBAAkB,CAAC;AAEhD;;GAEG;AACH,qBAAa,MAAM;IAIL,OAAO,CAAC,QAAQ,CAAC,YAAY;IAHzC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,YAAY,CAAyB;IAC7D,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,YAAY,CAAyB;gBAEhC,YAAY,EAAE,YAAY;IAEvD;;OAEG;IACH,MAAM,CAAC,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM;IAI9C;;OAEG;IACH,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,WAAW;IAuDnC;;;OAGG;IACH,OAAO,CAAC,kBAAkB;IA6B1B;;OAEG;IACH,OAAO,CAAC,eAAe;CAkBxB"}
+{"version":3,"file":"parser.d.ts","sourceRoot":"","sources":["../../src/lib/parser.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAU,WAAW,EAAY,MAAM,kBAAkB,CAAC;AACtE,OAAO,EAAE,YAAY,EAAE,MAAM,kBAAkB,CAAC;AAEhD;;GAEG;AACH,qBAAa,MAAM;IAIL,OAAO,CAAC,QAAQ,CAAC,YAAY;IAHzC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,YAAY,CAAyB;IAC7D,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,YAAY,CAAyB;gBAEhC,YAAY,EAAE,YAAY;IAEvD;;OAEG;IACH,MAAM,CAAC,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM;IAI9C;;OAEG;IACH,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM;IAI5C;;OAEG;IACH,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,WAAW;IAuDnC;;;OAGG;IACH,OAAO,CAAC,kBAAkB;IA6B1B;;OAEG;IACH,OAAO,CAAC,eAAe;CAkBxB"}

package/dist/lib/single-switcher.d.ts

@@ -8,6 +8,10 @@ export declare class SingleSwitcher extends Switcher {
      * Create a switcher for a specific file extension
      */
     static forExtension(extension: string): SingleSwitcher;
+    /**
+     * Create a switcher for a file path, correctly handling dotfiles like .env
+     */
+    static forFilePath(filePath: string): SingleSwitcher;
     /**
      * Switch to a specific option, deactivating all others
      */

package/dist/lib/single-switcher.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"single-switcher.d.ts","sourceRoot":"","sources":["../../src/lib/single-switcher.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAEjD,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAEzC;;GAEG;AACH,qBAAa,cAAe,SAAQ,QAAQ;IAC1C;;OAEG;IACH,MAAM,CAAC,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,cAAc;IAItD;;OAEG;IACH,MAAM,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM;CAyBxE"}
+{"version":3,"file":"single-switcher.d.ts","sourceRoot":"","sources":["../../src/lib/single-switcher.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAEjD,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAEzC;;GAEG;AACH,qBAAa,cAAe,SAAQ,QAAQ;IAC1C;;OAEG;IACH,MAAM,CAAC,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,cAAc;IAItD;;OAEG;IACH,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,MAAM,GAAG,cAAc;IAIpD;;OAEG;IACH,MAAM,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM;CAyBxE"}

package/dist/lib/types/comment-style.d.ts

@@ -21,6 +21,14 @@ export declare class CommentStyle {
      * Get comment style for a file extension
      */
     static fromExtension(extension: string): CommentStyle;
+    /**
+     * Get comment style for a file path, handling dotfiles like .env correctly.
+     *
+     * Unlike fromExtension(), this method properly handles:
+     * - `.env` files (where extname returns '')
+     * - `.env.local`, `.env.development`, etc. (where extname returns '.local', '.development')
+     */
+    static fromFilePath(filePath: string): CommentStyle;
     /**
      * Register a custom comment style for an extension
      */

package/dist/lib/types/comment-style.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"comment-style.d.ts","sourceRoot":"","sources":["../../../src/lib/types/comment-style.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,qBAAa,YAAY;IAiCrB,4CAA4C;aAC5B,WAAW,EAAE,MAAM;IAjCrC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,MAAM,CAsB5B;IAEF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,YAAY,CAA0B;IAE9D,2CAA2C;IAC3C,SAAgB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnC,0CAA0C;IAC1C,SAAgB,UAAU,CAAC,EAAE,MAAM,CAAC;;IAGlC,4CAA4C;IAC5B,WAAW,EAAE,MAAM,EACnC,SAAS,CAAC,EAAE,MAAM,EAClB,UAAU,CAAC,EAAE,MAAM;IAMrB;;OAEG;IACH,IAAI,gBAAgB,IAAI,OAAO,CAE9B;IAED;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY;IAKrD;;OAEG;IACH,MAAM,CAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,EAAE,KAAK,EAAE,YAAY,GAAG,IAAI;CAI9D"}
+{"version":3,"file":"comment-style.d.ts","sourceRoot":"","sources":["../../../src/lib/types/comment-style.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,qBAAa,YAAY;IAiCrB,4CAA4C;aAC5B,WAAW,EAAE,MAAM;IAjCrC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,MAAM,CAsB5B;IAEF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,YAAY,CAA0B;IAE9D,2CAA2C;IAC3C,SAAgB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnC,0CAA0C;IAC1C,SAAgB,UAAU,CAAC,EAAE,MAAM,CAAC;;IAGlC,4CAA4C;IAC5B,WAAW,EAAE,MAAM,EACnC,SAAS,CAAC,EAAE,MAAM,EAClB,UAAU,CAAC,EAAE,MAAM;IAMrB;;OAEG;IACH,IAAI,gBAAgB,IAAI,OAAO,CAE9B;IAED;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY;IAKrD;;;;;;OAMG;IACH,MAAM,CAAC,YAAY,CAAC,QAAQ,EAAE,MAAM,GAAG,YAAY;IAkBnD;;OAEG;IACH,MAAM,CAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,EAAE,KAAK,EAAE,YAAY,GAAG,IAAI;CAI9D"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lsst/pik-core",
-  "version": "0.6.5",
+  "version": "0.6.6",
   "description": "Core library for parsing and switching @pik config markers",
   "type": "module",
   "license": "MIT",