@minhpnq1807/contextos 0.1.6 → 0.1.8

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## 0.1.8
+
+- Limits automatic MCP telemetry wrapping to `code-review-graph` only.
+- Restores older ContextOS proxy wrappers on non-target MCP servers such as `agentmemory`.
+- Skips RTK-managed MCP commands instead of replacing their command entry with the ContextOS proxy.
+
+## 0.1.7
+
+- Filters documentation-only AGENTS entries such as MCP tool headings, HTML comments, generic "Key Tools" headings, and tool reference tables before scoring.
+- Keeps actionable tool instructions, for example `Use detect_changes for code review`, measurable through MCP telemetry.
+
 ## 0.1.6
 
 - Adds a transparent stdio MCP telemetry proxy that records `tools/call` events while forwarding requests to the original MCP server.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@minhpnq1807/contextos",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "Task-aware AGENTS.md context injection and compliance reporting for Codex.",
   "type": "module",
   "bin": {

package/plugins/ctx/lib/analyzer.js

@@ -62,6 +62,25 @@ const SYSTEM_USER_RULE_PATTERNS = [
   /\bminh_dev\b/i
 ];
 
+const DOCUMENTATION_HEADING_PATTERNS = [
+  /^mcp\s+tools?\s*:/i,
+  /^key\s+tools?$/i,
+  /^workflow$/i,
+  /^tools?$/i
+];
+
+const TOOL_REFERENCE_TOKENS = new Set([
+  "detect_changes",
+  "get_review_context",
+  "get_impact_radius",
+  "get_affected_flows",
+  "query_graph",
+  "semantic_search_nodes",
+  "get_architecture_overview",
+  "refactor_tool",
+  "list_communities"
+]);
+
 export function tokenize(value) {
   const normalized = String(value || "")
     .toLowerCase()
@@ -155,6 +174,7 @@ export function parseRules(markdown) {
 export function filterActionableRules(rules = []) {
   return rules
     .filter((rule) => !isSystemUserRule(rule))
+    .filter((rule) => !isDocumentationOnlyRule(rule))
     .map((rule, index) => ({ ...rule, id: `r${index + 1}`, originalOrder: index }));
 }
 
@@ -163,6 +183,32 @@ export function isSystemUserRule(rule) {
   return SYSTEM_USER_RULE_PATTERNS.some((pattern) => pattern.test(String(content || "")));
 }
 
+export function isDocumentationOnlyRule(rule) {
+  const content = String(typeof rule === "string" ? rule : rule?.content || "").trim();
+  const normalized = stripMarkdownEmphasis(content);
+  if (!normalized) return true;
+  if (/^<!--.*-->$/.test(normalized)) return true;
+  if (DOCUMENTATION_HEADING_PATTERNS.some((pattern) => pattern.test(normalized))) return true;
+  if (isMarkdownTableRule(normalized)) return true;
+  return false;
+}
+
+function stripMarkdownEmphasis(content) {
+  return String(content || "")
+    .replace(/^#+\s+/, "")
+    .replace(/^\*\*(.*)\*\*$/, "$1")
+    .trim();
+}
+
+function isMarkdownTableRule(content) {
+  if (!content.includes("|")) return false;
+  const pipeCount = (content.match(/\|/g) || []).length;
+  if (pipeCount < 4) return false;
+  const lower = content.toLowerCase();
+  const toolReferenceCount = [...TOOL_REFERENCE_TOKENS].filter((token) => lower.includes(token)).length;
+  return /\btool\b/.test(lower) && /\buse\s+when\b/.test(lower) && toolReferenceCount >= 2;
+}
+
 function dedupeRules(rules) {
   const seen = new Set();
   return rules.filter((rule) => {

package/plugins/ctx/lib/mcp-proxy-install.js

@@ -1,7 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
 
-const DEFAULT_TARGETS = new Set(["code-review-graph", "agentmemory"]);
+const DEFAULT_TARGETS = new Set(["code-review-graph"]);
 
 export function installMcpTelemetryProxies({ codexHome, marketplaceRoot, targets = DEFAULT_TARGETS } = {}) {
   const configPath = path.join(codexHome, "config.toml");
@@ -23,14 +23,29 @@ export function rewriteMcpTelemetryProxies(toml, { proxyPath, targets = DEFAULT_
   const skipped = [];
 
   for (const section of sections.reverse()) {
+    const body = lines.slice(section.start + 1, section.end);
+    const command = findStringValue(body, "command");
+    const args = findArrayValue(body, "args") || [];
+
+    if (command === "node" && args[0] === proxyPath && !targets.has(section.name)) {
+      const original = unwrapProxyArgs(args);
+      if (original) {
+        const nextBody = replaceOrInsertServerField(
+          replaceOrInsertServerField(body, "command", tomlString(original.command)),
+          "args",
+          tomlArray(original.args)
+        );
+        lines.splice(section.start + 1, section.end - section.start - 1, ...nextBody);
+        skipped.push({ name: section.name, reason: "unwrapped-non-target" });
+        continue;
+      }
+    }
+
     if (!targets.has(section.name)) {
       skipped.push({ name: section.name, reason: "not-targeted" });
       continue;
     }
 
-    const body = lines.slice(section.start + 1, section.end);
-    const command = findStringValue(body, "command");
-    const args = findArrayValue(body, "args") || [];
     if (!command) {
       skipped.push({ name: section.name, reason: "missing-command" });
       continue;
@@ -39,6 +54,10 @@ export function rewriteMcpTelemetryProxies(toml, { proxyPath, targets = DEFAULT_
       skipped.push({ name: section.name, reason: "already-wrapped" });
       continue;
     }
+    if (command === "rtk") {
+      skipped.push({ name: section.name, reason: "rtk-managed" });
+      continue;
+    }
 
     const nextBody = replaceOrInsertServerField(
       replaceOrInsertServerField(body, "command", tomlString("node")),
@@ -52,6 +71,15 @@ export function rewriteMcpTelemetryProxies(toml, { proxyPath, targets = DEFAULT_
   return { content: lines.join("\n"), wrapped: wrapped.reverse(), skipped: skipped.reverse() };
 }
 
+function unwrapProxyArgs(args) {
+  const separator = args.indexOf("--");
+  if (separator < 0 || separator >= args.length - 1) return null;
+  return {
+    command: args[separator + 1],
+    args: args.slice(separator + 2)
+  };
+}
+
 function findMcpServerSections(lines) {
   const sections = [];
   for (let index = 0; index < lines.length; index += 1) {