Merge branch 'main' into alert-autofix-3

Author: Sander-Toonen

.github/workflows/ci.yml

@@ -6,6 +6,9 @@ on:
   pull_request:
     branches: [ master, main ]
 
+permissions:
+  contents: read
+
 jobs:
   test:
     runs-on: ubuntu-latest

docs/syntax.md

@@ -99,7 +99,7 @@ The unary `+` and `-` operators are an exception, and always have their normal p
 | sqrt x   | Square root of x. Result is NaN (Not a Number) if x is negative. |
 | tan x    | Tangent of x (x is in radians) |
 | tanh x   | Hyperbolic tangent of x (x is in radians) |
-| trunc x  | Integral part of a X, looks like floor(x) unless for negative number |
+| trunc x  | Integral part of x; behaves like floor(x) for positive numbers, but rounds toward zero for negative numbers. |
 
 ## Pre-defined Functions
 
@@ -233,7 +233,7 @@ The parser includes comprehensive string manipulation capabilities.
 |:--------------------- |:----------- |
 | padLeft(str, len, padChar?)     | Pads a string on the left with spaces (or optional padding character) to reach the target length. |
 | padRight(str, len, padChar?)    | Pads a string on the right with spaces (or optional padding character) to reach the target length. |
-| padBoth(str, len, padChar?)     | Pads a string on both sides with spaces (or optional padding character) to reach the target length. If an odd number of padding characters is needed, the extra character is added on the right. |
+| padBoth(str, len, padChar?)     | Pads a string on both sides with spaces (or optional padding character) to reach the target length. If `len` is less than or equal to the string length, the original string is returned unchanged (no truncation). If an odd number of padding characters is needed, the extra character is added on the right. |
 
 ### Slicing and Encoding
 

package-lock.json

@@ -163,5 +163,11 @@
   "optionalDependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0",
     "zod": "^4.3.6"
+  },
+  "overrides": {
+    "hono": "^4.12.14"
+  },
+  "resolutions": {
+    "hono": "^4.12.14"
   }
 }

package.json

@@ -165,7 +165,6 @@ function loadExampleFromUrl() {
 (function() {
     const resizer = document.getElementById('verticalResizer');
     const contextPane = document.getElementById('contextPane');
-    const resultsPane = document.getElementById('resultsPane');
     const bottomArea = document.getElementById('bottomArea');
     let isResizing = false;
 
@@ -182,7 +181,6 @@ function loadExampleFromUrl() {
 
         const containerRect = bottomArea.getBoundingClientRect();
         const containerWidth = containerRect.width;
-        const resizerWidth = 6;
 
         let newLeftWidth = e.clientX - containerRect.left;
         newLeftWidth = Math.max(containerWidth * 0.2, Math.min(containerWidth * 0.8, newLeftWidth));

samples/language-service-sample/app.js

@@ -16,10 +16,7 @@ const GAMMA_P = [
   0.36899182659531622704e-5
 ] as const;
 
-function isInteger(value: number | undefined): boolean {
-  if (value === undefined) {
-    return false;
-  }
+function isInteger(value: number): boolean {
   return isFinite(value) && (value === Math.round(value));
 }
 

src/functions/math/advanced.ts

@@ -3,6 +3,8 @@ import { createLanguageService } from '../language-service/language-service.js';
 import type { LanguageServiceOptions } from '../language-service/language-service.types.js';
 import { registerTools } from './tools.js';
 
+const DEFAULT_MCP_SERVER_VERSION = '0.5.0';
+
 export interface CreateMcpServerOptions {
   operators?: LanguageServiceOptions['operators'];
   name?: string;
@@ -12,7 +14,7 @@ export interface CreateMcpServerOptions {
 export function createMcpServer(options: CreateMcpServerOptions = {}): McpServer {
   const server = new McpServer({
     name: options.name ?? 'expreszo-mcp',
-    version: options.version ?? '0.5.0'
+    version: options.version ?? DEFAULT_MCP_SERVER_VERSION
   });
 
   const ls = createLanguageService({ operators: options.operators });

src/mcp-server/server.ts

@@ -209,13 +209,13 @@ describe('Type Guards', () => {
       }
     });
 
-    it('should narrow types correctly with isFunction', () => {
+    it('should narrow types correctly with isFunction', async () => {
       const value: Value = Math.sin;
 
       if (isFunction(value)) {
         // TypeScript should know value is ExpressionFunction here
         expect(typeof value).toBe('function');
-        expect(typeof value(Math.PI / 2)).toBe('number');
+        expect(typeof await value(Math.PI / 2)).toBe('number');
       }
     });