bc-code-intelligence-mcp 1.5.9 → 1.6.0

package/embedded-knowledge/workflows/performance-audit.yaml

@@ -0,0 +1,148 @@
+type: "performance-audit"
+name: "Performance Audit"
+description: "Comprehensive performance analysis and optimization for Business Central AL code"
+specialist: "dean-debug"
+
+file_patterns:
+  - "**/*.al"
+file_exclusions:
+  - "**/test/**"
+  - "**/Test/**"
+  - "**/.altestrunner/**"
+
+phases:
+  - id: "inventory"
+    name: "File Inventory"
+    description: "Enumerate all AL files and assess codebase size"
+    required: true
+    mode: "autonomous"
+
+  - id: "analysis_quick"
+    name: "Quick Analysis"
+    description: "Rapid scan for common performance anti-patterns"
+    required: true
+    mode: "autonomous"
+    entry_conditions:
+      - "inventory phase completed"
+    available_actions:
+      - "scan_patterns"
+      - "identify_hotspots"
+
+  - id: "analysis_full"
+    name: "Full Analysis"
+    description: "Deep analysis of all procedures for performance patterns"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "analysis_quick phase completed"
+    available_actions:
+      - "analyze_procedure"
+      - "check_all_patterns"
+      - "document_coverage"
+      - "cross_reference"
+
+  - id: "performance_quick"
+    name: "Quick Optimization"
+    description: "Apply straightforward optimizations (SetLoadFields, obvious SIFT)"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "analysis_full phase completed"
+    available_actions:
+      - "apply_setloadfields"
+      - "apply_sift"
+      - "optimize_loops"
+
+  - id: "performance_full"
+    name: "Full Optimization"
+    description: "Complex optimizations requiring architectural consideration"
+    required: false
+    mode: "agent_driven"
+    entry_conditions:
+      - "performance_quick phase completed"
+    available_actions:
+      - "redesign_query"
+      - "implement_caching"
+      - "batch_operations"
+      - "refactor_loops"
+
+  - id: "execution_validation"
+    name: "Execution Validation"
+    description: "Verify optimizations are correctly implemented and functional"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "performance_quick phase completed"
+    available_actions:
+      - "verify_changes"
+      - "test_compilation"
+      - "validate_logic"
+
+  - id: "verification"
+    name: "Coverage Verification"
+    description: "Ensure all files analyzed and no procedures missed"
+    required: true
+    mode: "autonomous"
+    entry_conditions:
+      - "execution_validation phase completed"
+    available_actions:
+      - "validate_coverage"
+      - "detect_gaps"
+      - "generate_report"
+
+per_file_checklist:
+  - id: "scan"
+    type: "analysis"
+    description: "Scan for performance patterns"
+    required: true
+
+  - id: "setloadfields_check"
+    type: "topic_application"
+    description: "Check SetLoadFields before record operations"
+    required: true
+    conditions:
+      content_pattern: "Find(Set|First|Last)\\s*\\("
+
+  - id: "sift_check"
+    type: "topic_application"
+    description: "Check for manual summation that should use SIFT"
+    required: true
+    conditions:
+      content_pattern: "repeat.*\\+.*until|:=.*\\+.*;"
+
+  - id: "loop_check"
+    type: "topic_application"
+    description: "Check loop efficiency patterns"
+    required: true
+    conditions:
+      content_pattern: "repeat"
+
+  - id: "calcsums_check"
+    type: "topic_application"
+    description: "Verify CalcSums/CalcFields usage"
+    required: true
+    conditions:
+      content_pattern: "CalcSums|CalcFields"
+
+  - id: "deleteall_check"
+    type: "topic_application"
+    description: "Review DeleteAll usage and trigger implications"
+    required: true
+    conditions:
+      content_pattern: "DeleteAll"
+
+  - id: "optimization_complete"
+    type: "validation"
+    description: "All performance issues addressed"
+    required: true
+
+topic_discovery:
+  enabled: true
+  tool: "analyze_al_code"
+  auto_expand_checklist: true
+  min_relevance_score: 0.5
+
+completion_rules:
+  require_all_files: true
+  require_all_checklist_items: true
+  allow_skip_with_reason: true

package/embedded-knowledge/workflows/proposal-review.yaml

@@ -0,0 +1,117 @@
+type: "proposal-review"
+name: "Proposal Review Workflow"
+description: "Systematic approach to AI-assisted development that builds trust through transparent reasoning, reviewable proposals, and collaborative specialist validation"
+specialist: "parker-pragmatic"
+
+file_patterns:
+  - "**/*.al"
+  - "**/*.md"
+file_exclusions:
+  - "**/node_modules/**"
+  - "**/.parker-proposals/**"
+
+phases:
+  - id: "understanding"
+    name: "Understanding & Reasoning"
+    description: "Establish clear understanding of the request, explain AI reasoning, identify alternatives, state confidence level, recommend specialists"
+    required: true
+    mode: "guided"
+    available_actions:
+      - "restate_request"
+      - "explain_reasoning"
+      - "identify_alternatives"
+      - "state_confidence"
+      - "recommend_specialists"
+
+  - id: "proposal_creation"
+    name: "Proposal Creation"
+    description: "Generate complete, reviewable implementation in temporary sandbox location (.parker-proposals/)"
+    required: true
+    mode: "agent_driven"
+    entry_conditions:
+      - "understanding phase completed"
+    available_actions:
+      - "create_sandbox"
+      - "implement_solution"
+      - "add_documentation"
+      - "provide_comparison"
+      - "create_cleanup_instructions"
+
+  - id: "specialist_review"
+    name: "Specialist Review"
+    description: "Coordinate validation from relevant BC specialists before developer decides"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "proposal_creation phase completed"
+    available_actions:
+      - "identify_reviewers"
+      - "present_proposal"
+      - "gather_feedback"
+      - "synthesize_input"
+      - "revise_proposal"
+
+  - id: "developer_decision"
+    name: "Developer Decision"
+    description: "Present complete picture, explain trade-offs, respect developer choice, support implementation"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "specialist_review phase completed"
+    available_actions:
+      - "present_options"
+      - "explain_tradeoffs"
+      - "accept_proposal"
+      - "modify_proposal"
+      - "reject_proposal"
+
+  - id: "verification_cleanup"
+    name: "Verification & Cleanup"
+    description: "Confirm results match expectations and remove all temporary artifacts"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "developer_decision phase completed"
+    available_actions:
+      - "verify_implementation"
+      - "validate_assumptions"
+      - "check_performance"
+      - "remove_proposals"
+      - "document_learning"
+
+per_file_checklist:
+  - id: "understand_context"
+    type: "analysis"
+    description: "Analyze file context and identify relevant patterns"
+    required: true
+
+  - id: "create_proposal"
+    type: "custom"
+    description: "Create proposal with reasoning and alternatives"
+    required: true
+
+  - id: "specialist_validation"
+    type: "custom"
+    description: "Get specialist validation on proposed changes"
+    required: true
+
+  - id: "decision_recorded"
+    type: "validation"
+    description: "Developer decision recorded (accept/modify/reject)"
+    required: true
+
+  - id: "cleanup_complete"
+    type: "validation"
+    description: "All temporary artifacts removed"
+    required: true
+
+topic_discovery:
+  enabled: true
+  tool: "find_bc_knowledge"
+  auto_expand_checklist: false
+  min_relevance_score: 0.5
+
+completion_rules:
+  require_all_files: false
+  require_all_checklist_items: true
+  allow_skip_with_reason: true

package/embedded-knowledge/workflows/security-audit.yaml

@@ -0,0 +1,139 @@
+type: "security-audit"
+name: "Security Audit"
+description: "Security review of AL code focusing on permissions, input validation, and data protection"
+specialist: "seth-security"
+
+file_patterns:
+  - "**/*.al"
+file_exclusions:
+  - "**/test/**"
+  - "**/Test/**"
+  - "**/.altestrunner/**"
+
+phases:
+  - id: "inventory"
+    name: "File Inventory"
+    description: "Enumerate all AL files in scope"
+    required: true
+    mode: "autonomous"
+
+  - id: "permission_scan"
+    name: "Permission Analysis"
+    description: "Scan for permission-related code and validate permission sets"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "inventory phase completed"
+    available_actions:
+      - "scan_permissions"
+      - "validate_permission_sets"
+      - "check_inherent_permissions"
+
+  - id: "input_validation"
+    name: "Input Validation Review"
+    description: "Check for proper validation of user inputs and external data"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "inventory phase completed"
+    available_actions:
+      - "check_user_inputs"
+      - "check_api_inputs"
+      - "check_file_inputs"
+
+  - id: "data_protection"
+    name: "Data Protection Review"
+    description: "Review handling of sensitive data, credentials, and secrets"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "inventory phase completed"
+    available_actions:
+      - "check_credentials"
+      - "check_pii_handling"
+      - "check_encryption"
+
+  - id: "temporary_table_review"
+    name: "Temporary Table Safety"
+    description: "Verify IsTemporary checks before bulk operations"
+    required: true
+    mode: "guided"
+    entry_conditions:
+      - "inventory phase completed"
+    available_actions:
+      - "check_istemporary"
+      - "check_deleteall_safety"
+      - "check_modifyall_safety"
+
+  - id: "api_security"
+    name: "API Security Review"
+    description: "Review API endpoints for proper authentication and authorization"
+    required: false
+    mode: "guided"
+    entry_conditions:
+      - "permission_scan phase completed"
+    available_actions:
+      - "check_api_authentication"
+      - "check_api_authorization"
+      - "check_rate_limiting"
+
+  - id: "summary"
+    name: "Security Report"
+    description: "Generate security findings report with risk ratings"
+    required: true
+    mode: "autonomous"
+    entry_conditions:
+      - "permission_scan phase completed"
+      - "input_validation phase completed"
+      - "data_protection phase completed"
+      - "temporary_table_review phase completed"
+
+per_file_checklist:
+  - id: "scan"
+    type: "analysis"
+    description: "Scan for security patterns"
+    required: true
+
+  - id: "permission_check"
+    type: "topic_application"
+    description: "Verify proper permission handling"
+    required: true
+    conditions:
+      content_pattern: "Permissions|InherentPermissions|TestPermissions"
+
+  - id: "input_validation_check"
+    type: "topic_application"
+    description: "Check input validation patterns"
+    required: true
+    conditions:
+      content_pattern: "Validate|TestField|FieldError"
+
+  - id: "istemporary_check"
+    type: "topic_application"
+    description: "Verify IsTemporary safeguards on bulk operations"
+    required: true
+    conditions:
+      content_pattern: "DeleteAll|ModifyAll"
+
+  - id: "credential_check"
+    type: "topic_application"
+    description: "Check for hardcoded credentials or secrets"
+    required: true
+    conditions:
+      content_pattern: "password|secret|token|apikey|credential"
+
+  - id: "security_complete"
+    type: "validation"
+    description: "Security review complete for this file"
+    required: true
+
+topic_discovery:
+  enabled: true
+  tool: "analyze_al_code"
+  auto_expand_checklist: true
+  min_relevance_score: 0.7
+
+completion_rules:
+  require_all_files: true
+  require_all_checklist_items: true
+  allow_skip_with_reason: true

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bc-code-intelligence-mcp",
-  "version": "1.5.9",
+  "version": "1.6.0",
   "description": "BC Code Intelligence MCP Server - Complete Specialist Bundle with AI-driven expert consultation, seamless handoffs, and context-preserving workflows",
   "main": "dist/index.js",
   "type": "module",
@@ -9,7 +9,6 @@
     "embedded-knowledge/"
   ],
   "bin": {
-    "bc-code-intel": "dist/cli/bc-code-intel-cli.js",
     "bc-code-intel-server": "dist/index.js",
     "bc-code-intelligence-mcp": "dist/index.js"
   },
@@ -52,6 +51,9 @@
     "marked": "^11.1.1",
     "simple-git": "^3.28.0",
     "tslib": "^2.8.1",
+    "wink-bm25-text-search": "^3.0.0",
+    "wink-eng-lite-web-model": "^1.0.0",
+    "wink-nlp": "^2.0.0",
     "yaml": "^2.3.4",
     "zod": "^3.25.76"
   },

package/dist/cache/cache-manager.d.ts

@@ -1,95 +0,0 @@
-/**
- * Advanced Cache Manager with Multi-Level Caching
- *
- * Implements intelligent caching for topics, search results, and layer data
- * with TTL, LRU eviction, memory pressure handling, and performance analytics.
- */
-import { AtomicTopic, TopicSearchResult } from '../types/bc-knowledge.js';
-import { CacheSettings, CacheTTLSettings } from '../types/index.js';
-export interface CacheStats {
-    hits: number;
-    misses: number;
-    evictions: number;
-    total_requests: number;
-    hit_rate: number;
-    memory_usage_bytes: number;
-    cache_size: number;
-}
-export interface CacheEntry<T> {
-    data: T;
-    timestamp: number;
-    ttl: number;
-    access_count: number;
-    last_accessed: number;
-    size_bytes: number;
-}
-export declare class AdvancedCacheManager {
-    private topicCache;
-    private searchCache;
-    private layerCache;
-    private stats;
-    private readonly config;
-    private cleanupInterval?;
-    constructor(config: CacheSettings);
-    /**
-     * Cache a topic with intelligent TTL based on source type
-     */
-    cacheTopic(key: string, topic: AtomicTopic, sourceType: keyof CacheTTLSettings): void;
-    /**
-     * Retrieve topic from cache with hit tracking
-     */
-    getTopic(key: string): AtomicTopic | null;
-    /**
-     * Cache search results with query-based invalidation
-     */
-    cacheSearchResults(query: string, results: TopicSearchResult[], ttl?: number): void;
-    /**
-     * Retrieve cached search results
-     */
-    getSearchResults(query: string): TopicSearchResult[] | null;
-    /**
-     * Cache layer data (indexes, metadata, etc.)
-     */
-    cacheLayerData(layerName: string, key: string, data: any, sourceType: keyof CacheTTLSettings): void;
-    /**
-     * Retrieve cached layer data
-     */
-    getLayerData(layerName: string, key: string): any | null;
-    /**
-     * Invalidate cache entries for a specific layer
-     */
-    invalidateLayer(layerName: string): number;
-    /**
-     * Warm up cache with frequently accessed topics
-     */
-    warmUpCache(topics: Array<{
-        key: string;
-        topic: AtomicTopic;
-        sourceType: keyof CacheTTLSettings;
-    }>): Promise<void>;
-    /**
-     * Preload critical cache entries based on usage patterns
-     */
-    preloadCriticalPaths(criticalTopics: string[]): Promise<void>;
-    /**
-     * Get comprehensive cache statistics
-     */
-    getStats(): CacheStats;
-    /**
-     * Clear all caches
-     */
-    clearAll(): void;
-    /**
-     * Shutdown and cleanup
-     */
-    shutdown(): void;
-    private getTTLForSourceType;
-    private isExpired;
-    private generateSearchKey;
-    private estimateSize;
-    private calculateTotalMemoryUsage;
-    private enforceMemoryLimits;
-    private startPeriodicCleanup;
-    private cleanupExpiredEntries;
-}
-//# sourceMappingURL=cache-manager.d.ts.map

package/dist/cache/cache-manager.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"cache-manager.d.ts","sourceRoot":"","sources":["../../src/cache/cache-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,WAAW,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAC1E,OAAO,EAAE,aAAa,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AAEpE,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;IAClB,cAAc,EAAE,MAAM,CAAC;IACvB,QAAQ,EAAE,MAAM,CAAC;IACjB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,UAAU,CAAC,CAAC;IAC3B,IAAI,EAAE,CAAC,CAAC;IACR,SAAS,EAAE,MAAM,CAAC;IAClB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,qBAAa,oBAAoB;IAC/B,OAAO,CAAC,UAAU,CAA8C;IAChE,OAAO,CAAC,WAAW,CAAsD;IACzE,OAAO,CAAC,UAAU,CAAsC;IAExD,OAAO,CAAC,KAAK,CAKX;IAEF,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAgB;IACvC,OAAO,CAAC,eAAe,CAAC,CAAiB;gBAE7B,MAAM,EAAE,aAAa;IAKjC;;OAEG;IACH,UAAU,CAAC,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,EAAE,UAAU,EAAE,MAAM,gBAAgB,GAAG,IAAI;IAerF;;OAEG;IACH,QAAQ,CAAC,GAAG,EAAE,MAAM,GAAG,WAAW,GAAG,IAAI;IAwBzC;;OAEG;IACH,kBAAkB,CAAC,KAAK,EAAE,MAAM,EAAE,OAAO,EAAE,iBAAiB,EAAE,EAAE,GAAG,GAAE,MAAsB,GAAG,IAAI;IAelG;;OAEG;IACH,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,iBAAiB,EAAE,GAAG,IAAI;IAc3D;;OAEG;IACH,cAAc,CAAC,SAAS,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,EAAE,UAAU,EAAE,MAAM,gBAAgB,GAAG,IAAI;IAiBnG;;OAEG;IACH,YAAY,CAAC,SAAS,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,IAAI;IAcxD;;OAEG;IACH,eAAe,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM;IA2B1C;;OAEG;IACG,WAAW,CAAC,MAAM,EAAE,KAAK,CAAC;QAAE,GAAG,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,WAAW,CAAC;QAAC,UAAU,EAAE,MAAM,gBAAgB,CAAA;KAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAqBxH;;OAEG;IACG,oBAAoB,CAAC,cAAc,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;IAUnE;;OAEG;IACH,QAAQ,IAAI,UAAU;IAiBtB;;OAEG;IACH,QAAQ,IAAI,IAAI;IAUhB;;OAEG;IACH,QAAQ,IAAI,IAAI;IAShB,OAAO,CAAC,mBAAmB;IAqB3B,OAAO,CAAC,SAAS;IAOjB,OAAO,CAAC,iBAAiB;IAKzB,OAAO,CAAC,YAAY;IAKpB,OAAO,CAAC,yBAAyB;IAkBjC,OAAO,CAAC,mBAAmB;IAuC3B,OAAO,CAAC,oBAAoB;IAO5B,OAAO,CAAC,qBAAqB;CA+B9B"}