fix: Some issues where the make precommit check fails

Signed-off-by: plotor <zhenchao.wang@hotmail.com>

Author: plotor

.github/workflows/python-tests.yml

@@ -86,14 +86,17 @@ jobs:
         pip install dist/*.tar.gz
         pip uninstall -y memoryos
 
-    # Ruff checks
-    - name: Install test group dependencies
+    # Ruff and Pre-commit checks
+    - name: Install test and dev group dependencies
       run: |
-        poetry install --no-interaction --with test
+        poetry install --no-interaction --with test --with dev
     - name: Ruff checks
       run: |
         poetry run ruff check
         poetry run ruff format --check
+    - name: Pre-commit checks
+      run: |
+        poetry run pre-commit run -a
 
     # PyTest checks
     - name: Install all extra dependencies

.pre-commit-config.yaml

@@ -12,6 +12,7 @@ repos:
       - id: check-toml
         exclude: tests/fixtures/invalid_lock/poetry\.lock
       - id: check-yaml
+        exclude: ^deploy/helm/templates/
       - id: pretty-format-json
         args: [--autofix, --no-ensure-ascii, --no-sort-keys]
       - id: check-ast

Makefile

@@ -35,6 +35,9 @@ format:
 pre_commit:
 	poetry run pre-commit run -a
 
+# Alias for pre_commit
+precommit: pre_commit
+
 serve:
 	poetry run uvicorn memos.api.server_api:app
 

apps/MemOS-Cloud-OpenClaw-Plugin/lib/check-update.js

@@ -95,30 +95,30 @@ function compareVersions(v1, v2) {
   const split2 = v2.split("-");
   const parts1 = split1[0].split(".").map(Number);
   const parts2 = split2[0].split(".").map(Number);
-  
+
   // Compare major.minor.patch
   for (let i = 0; i < Math.max(parts1.length, parts2.length); i++) {
     const p1 = parts1[i] || 0;
     const p2 = parts2[i] || 0;
     if (p1 > p2) return 1;
     if (p1 < p2) return -1;
   }
-  
+
   // If base versions are equal, compare pre-release tags.
   // A version WITH a pre-release tag is LOWER than a version WITHOUT one.
   // e.g. 0.1.8-beta is less than 0.1.8. 0.1.8 is the final release.
   const hasPre1 = split1.length > 1;
   const hasPre2 = split2.length > 1;
-  
+
   if (hasPre1 && !hasPre2) return -1; // v1 is a beta, v2 is a full release
   if (!hasPre1 && hasPre2) return 1;  // v1 is a full release, v2 is a beta
   if (!hasPre1 && !hasPre2) return 0; // both are full releases and equal
-  
+
   // If both are pre-releases, do a basic string compare on the tag
   // "alpha" < "beta" < "rc"
   if (split1[1] > split2[1]) return 1;
   if (split1[1] < split2[1]) return -1;
-  
+
   return 0;
 }
 
@@ -202,7 +202,7 @@ export function startUpdateChecker(log) {
       child.stdout.on("data", (data) => {
         const outText = data.toString();
         log.info?.(`${ANSI.CYAN}[${cliName}-cli]${ANSI.RESET}\n${outText.trim()}`);
-        
+
         // Auto-reply to any [y/N] prompts from the CLI
         if (outText.toLowerCase().includes("[y/n]")) {
           child.stdin.write("y\n");
@@ -212,7 +212,7 @@ export function startUpdateChecker(log) {
       child.stderr.on("data", (data) => {
         const errText = data.toString();
         log.warn?.(`${ANSI.RED}[${cliName}-cli]${ANSI.RESET}\n${errText.trim()}`);
-        
+
         // Some CLIs output interactive prompts to stderr instead of stdout
         if (errText.toLowerCase().includes("[y/n]")) {
           child.stdin.write("y\n");

apps/MemOS-Cloud-OpenClaw-Plugin/scripts/sync-version.js

@@ -20,11 +20,11 @@ const filesToUpdate = [
 
 filesToUpdate.forEach(fileName => {
   const filePath = path.resolve(__dirname, '..', fileName);
-  
+
   if (fs.existsSync(filePath)) {
     try {
       const content = JSON.parse(fs.readFileSync(filePath, 'utf8'));
-      
+
       if (content.version !== newVersion) {
         content.version = newVersion;
         // Write back with 2 spaces indentation and a newline at the end

apps/MemOS-Cloud-OpenClaw-Plugin/test/query-strip.test.mjs

@@ -162,16 +162,16 @@ test("supports leading blank lines before inbound metadata", () => {
 });
 
 test("strips Feishu injected prompt", () => {
-  const input = `System: [2026-03-17 14:17:33 GMT+8] Feishu[default] DM from ou_37e8a1514c24e8afd9cfeca86f679980: 我叫什么名字 
- 
- Conversation info (untrusted metadata): 
- \`\`\`json 
- { 
-  "timestamp": "Tue 2026-03-17 14:17 GMT+8" 
- } 
- \`\`\` 
- 
- [message_id: om_x100b54bb510590dcc2998da17ca2c2b] 
+  const input = `System: [2026-03-17 14:17:33 GMT+8] Feishu[default] DM from ou_37e8a1514c24e8afd9cfeca86f679980: 我叫什么名字
+
+ Conversation info (untrusted metadata):
+ \`\`\`json
+ {
+  "timestamp": "Tue 2026-03-17 14:17 GMT+8"
+ }
+ \`\`\`
+
+ [message_id: om_x100b54bb510590dcc2998da17ca2c2b]
  ou_37e8a1514c24e8afd9cfeca86f679980: 我叫什么名字 `;
 
   assert.equal(stripOpenClawInjectedPrefix(input), "我叫什么名字");
@@ -191,7 +191,7 @@ ou_real: actual message`;
 
 test("strips Feishu prompt without system header", () => {
   const input = `
-[message_id: om_x100b54bb510590dcc2998da17ca2c2b] 
+[message_id: om_x100b54bb510590dcc2998da17ca2c2b]
 ou_37e8a1514c24e8afd9cfeca86f679980: 我叫什么名字 `;
   assert.equal(stripOpenClawInjectedPrefix(input), "我叫什么名字");
 });

apps/memos-local-openclaw/openclaw.plugin.json

@@ -33,5 +33,7 @@
       "If better-sqlite3 fails to build, ensure you have C++ build tools: xcode-select --install (macOS) or build-essential (Linux)"
     ]
   },
-  "extensions": ["./index.ts"]
+  "extensions": [
+    "./index.ts"
+  ]
 }

apps/memos-local-openclaw/skill/browserwing-executor/SKILL.md

@@ -507,4 +507,3 @@ All operations return:
   "detail": "Detailed error message"
 }
 ```
-

apps/memos-local-openclaw/src/sharing/types.ts

@@ -99,4 +99,3 @@ export interface SkillBundle {
   metadata: SkillBundleMetadata;
   bundle: SkillGenerateOutput;
 }
-

apps/memos-local-openclaw/src/viewer/html.ts

@@ -1411,7 +1411,7 @@ input,textarea,select{font-family:inherit;font-size:inherit}
         <h3><span class="icon">\u{1F4CA}</span> <span data-i18n="chart.writes">Memory Writes per Day</span></h3>
         <div class="chart-bars" id="chartWrites"></div>
       </div>
-      
+
       <div class="analytics-section" id="toolPerfSection" style="position:relative">
         <div style="display:flex;align-items:center;justify-content:space-between;margin-bottom:20px">
           <h3 style="margin-bottom:0"><span class="icon">\u26A1</span> <span data-i18n="chart.toolperf">Tool Response Time</span> <span style="font-size:10px;color:var(--text-muted);font-weight:500;text-transform:none;letter-spacing:0;margin-left:4px">(per minute avg)</span></h3>
@@ -1443,7 +1443,7 @@ input,textarea,select{font-family:inherit;font-size:inherit}
           <select id="logToolFilter" onchange="onLogFilterChange()" style="font-size:12px;padding:4px 8px;border-radius:6px;border:1px solid var(--border);background:var(--card);color:var(--text);min-width:120px">
             <option value="" data-i18n="logs.allTools">All Tools</option>
           </select>
-          
+
         </div>
         <div class="logs-toolbar-right">
           <input type="checkbox" id="logAutoRefresh" style="display:none">
@@ -6722,7 +6722,7 @@ async function loadConfig(){
     const embProv=(cfg.embedding||{}).provider;
     const sumProv=(cfg.summarizer||{}).provider;
     const skProv=((cfg.skillEvolution||{}).summarizer||{}).provider;
-    
+
 
     document.getElementById('cfgSharingEnabled').checked=!!sharing.enabled;
     _sharingRole=sharing.role||'client';
@@ -7347,7 +7347,7 @@ function renderToolChart(data){
       return '<linearGradient id="tg'+i+'" x1="0" y1="0" x2="0" y2="1"><stop offset="0" stop-color="'+c+'" stop-opacity=".08"/><stop offset="1" stop-color="'+c+'" stop-opacity="0"/></linearGradient>'+
         '';
     }).join('')+'</defs>'+
-    
+
     gridHtml+labelsHtml+pathsHtml+dotsHtml+
     '<line class="crosshair" x1="0" y1="'+pad.t+'" x2="0" y2="'+(pad.t+ch)+'" stroke="var(--text-muted)" stroke-width="0.5" stroke-dasharray="3 3" opacity="0" />'+
     '<rect class="hover-rect" x="'+pad.l+'" y="'+pad.t+'" width="'+cw+'" height="'+ch+'" fill="transparent" />'+