feat: add comprehensive repository badges and improvements #37
Conversation
- Replace .eslintrc.js with eslint.config.js (ESLint v9 requirement) - Add 'type': 'module' to package.json for ES module support - Update Jest config to use ES module syntax - Fix rollup configuration for new @rollup/plugin-typescript version - Add proper globals configuration for Node.js and browser APIs - Install globals package for ESLint configuration - All tests and linting now working with new versions Resolves ESLint v9 migration issues from Renovate updates
- Add comment to TypeScript README to trigger CI - Test if ESLint v9 configuration works in CI environment
📝 WalkthroughWalkthroughReplaces legacy ESLint files with a new flat Changes
Sequence Diagram(s)Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Pre-merge checks and finishing touches❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
🧰 Additional context used📓 Path-based instructions (6)clients/typescript/**📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Files:
⚙️ CodeRabbit configuration file
Files:
**/*📄 CodeRabbit inference engine (.cursor/rules/security.mdc)
Files:
clients/**/*📄 CodeRabbit inference engine (CONTRIBUTING.md)
Files:
clients/typescript/**/*.{ts,tsx,js,jsx}📄 CodeRabbit inference engine (CONTRIBUTING.md)
Files:
clients/typescript/**/package.json📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Files:
clients/typescript/package.json📄 CodeRabbit inference engine (.cursor/rules/typescript.mdc)
Files:
🧠 Learnings (6)📓 Common learnings📚 Learning: 2025-10-23T13:33:13.407ZApplied to files:
📚 Learning: 2025-10-23T13:31:29.892ZApplied to files:
📚 Learning: 2025-10-23T13:26:37.198ZApplied to files:
📚 Learning: 2025-10-23T13:26:37.198ZApplied to files:
📚 Learning: 2025-10-23T13:31:29.892ZApplied to files:
🔇 Additional comments (7)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
- Convert CommonJS require() to ES module import syntax - Add __dirname and __filename polyfills for ES modules - Fixes TypeScript client CI failure due to ES module configuration
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (2)
clients/typescript/README.mdis excluded by!**/*.mdand included byclients/**clients/typescript/package-lock.jsonis excluded by!**/package-lock.jsonand included byclients/**
📒 Files selected for processing (9)
clients/typescript/.eslintignore(0 hunks)clients/typescript/.eslintrc.js(0 hunks)clients/typescript/eslint.config.js(1 hunks)clients/typescript/jest.config.js(1 hunks)clients/typescript/package.json(2 hunks)clients/typescript/rollup.config.js(1 hunks)clients/typescript/src/index.ts(9 hunks)clients/typescript/tests/contextforgeClient.test.ts(14 hunks)clients/typescript/tests/setup.ts(1 hunks)
💤 Files with no reviewable changes (2)
- clients/typescript/.eslintignore
- clients/typescript/.eslintrc.js
🧰 Additional context used
📓 Path-based instructions (10)
clients/typescript/**
📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Update TypeScript client types and examples to match API changes
TypeScript client should use Husky with fast pre-commit checks and smoke-test pre-push workflow
Files:
clients/typescript/eslint.config.jsclients/typescript/tests/contextforgeClient.test.tsclients/typescript/jest.config.jsclients/typescript/rollup.config.jsclients/typescript/tests/setup.tsclients/typescript/package.jsonclients/typescript/src/index.ts
⚙️ CodeRabbit configuration file
clients/typescript/**: Ensure strict typing, accurate DTOs from OpenAPI, consistent error shapes, and robust timeout/retry semantics.
Prefer fetch/axios configurations with sane defaults; avoid throwing ambiguous any-typed errors.
Files:
clients/typescript/eslint.config.jsclients/typescript/tests/contextforgeClient.test.tsclients/typescript/jest.config.jsclients/typescript/rollup.config.jsclients/typescript/tests/setup.tsclients/typescript/package.jsonclients/typescript/src/index.ts
**/*
📄 CodeRabbit inference engine (.cursor/rules/security.mdc)
Avoid committing content matching common secret patterns (e.g., sk-..., AKIA..., ghp_..., password='...')
Files:
clients/typescript/eslint.config.jsclients/typescript/tests/contextforgeClient.test.tsclients/typescript/jest.config.jsclients/typescript/rollup.config.jsclients/typescript/tests/setup.tsclients/typescript/package.jsonclients/typescript/src/index.ts
clients/**/*
📄 CodeRabbit inference engine (CONTRIBUTING.md)
clients/**/*: Client libraries should follow sound API client design patterns
Client libraries must implement error handling and retry logic where appropriate
Ensure type safety and clear interfaces in client libraries
Provide documentation and examples for client libraries
Maintain cross-platform compatibility in client libraries
Files:
clients/typescript/eslint.config.jsclients/typescript/tests/contextforgeClient.test.tsclients/typescript/jest.config.jsclients/typescript/rollup.config.jsclients/typescript/tests/setup.tsclients/typescript/package.jsonclients/typescript/src/index.ts
clients/typescript/**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Enforce ESLint + Prettier standards in the TypeScript client
Files:
clients/typescript/eslint.config.jsclients/typescript/tests/contextforgeClient.test.tsclients/typescript/jest.config.jsclients/typescript/rollup.config.jsclients/typescript/tests/setup.tsclients/typescript/src/index.ts
clients/typescript/**/*.ts
📄 CodeRabbit inference engine (.cursor/rules/typescript.mdc)
clients/typescript/**/*.ts: Use strict typing with interfaces for request/response objects (e.g., MemoryItem, StoreRequest, StoreResponse) and avoid weakly typed functions
Generate DTOs from the OpenAPI spec (e.g., using openapi-typescript) instead of writing manual DTOs to prevent drift
Avoid any; prefer specific types or unknown with type guards for processing API data
Implement a consistent ApiError shape and a base ContextForgeError class for API errors
Use typed exception subclasses (AuthenticationError, ValidationError, RateLimitError) and branch on them explicitly
Map technical error codes to user-friendly messages via a central ERROR_MESSAGES map and a helper (getUserFriendlyMessage)
Implement retry logic with configurable RetryConfig (maxRetries, baseDelay, maxDelay, jitter) and exponential backoff
Only retry on transient conditions (network errors, rate limits, server errors) via a shouldRetry predicate
HTTP client should use fetch with sane defaults: baseUrl, headers, JSON body, timeout via AbortController, and throw on non-OK responses
Use keep-alive/connection pooling where supported (e.g., Connection: keep-alive) to reuse connections
Support request/response interceptors to transform requests and responses centrally
Use zod for runtime validation of requests and responses; throw ValidationError on invalid data
Validate inputs: namespaces (non-empty, <=100, /^[a-zA-Z0-9_-]+$/), text (non-empty, <=65536), items array (1..100)
Code quality: No any types used in the codebase
Code quality: All functions declare explicit return types
Code quality: Timeouts are configured for HTTP requests (e.g., via AbortController)
Code quality: Public types are exported properly (e.g., DTOs, client interfaces)
Files:
clients/typescript/tests/contextforgeClient.test.tsclients/typescript/tests/setup.tsclients/typescript/src/index.ts
clients/typescript/**/*.{test,spec}.ts
📄 CodeRabbit inference engine (.cursor/rules/typescript.mdc)
clients/typescript/**/*.{test,spec}.ts: Write tests using Jest/Vitest conventions; mock fetch; assert request shape and response handling
Provide integration-style tests covering full workflows (store, search) using mocks as needed
Code quality: Retry logic is covered by tests (e.g., verifies attempts, delays, shouldRetry behavior)
Files:
clients/typescript/tests/contextforgeClient.test.ts
clients/typescript/{**/*.test.ts,**/*.test.tsx,**/*.spec.ts,**/*.spec.tsx,**/*.test.js,**/*.test.jsx,**/*.spec.js,**/*.spec.jsx}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Use Jest tests with a smoke-test subset optimization for the TypeScript client
Files:
clients/typescript/tests/contextforgeClient.test.ts
clients/typescript/rollup.config.js
📄 CodeRabbit inference engine (.cursor/rules/typescript.mdc)
Use Rollup to build ESM, CJS, and minified UMD bundles with @rollup/plugin-typescript (declarations off in bundling) and terser for minification
Files:
clients/typescript/rollup.config.js
clients/typescript/**/package.json
📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Bump TypeScript client version in package.json for breaking changes
Files:
clients/typescript/package.json
clients/typescript/package.json
📄 CodeRabbit inference engine (.cursor/rules/typescript.mdc)
package.json should expose proper entry points (main/module/types/exports) and include build/test/lint scripts (rollup, tsc, vitest, eslint)
Files:
clients/typescript/package.json
🧠 Learnings (8)
📓 Common learnings
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: CONTRIBUTING.md:0-0
Timestamp: 2025-10-23T13:33:13.407Z
Learning: Applies to clients/typescript/**/*.{ts,tsx,js,jsx} : Enforce ESLint + Prettier standards in the TypeScript client
📚 Learning: 2025-10-23T13:33:13.407Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: CONTRIBUTING.md:0-0
Timestamp: 2025-10-23T13:33:13.407Z
Learning: Applies to clients/typescript/**/*.{ts,tsx,js,jsx} : Enforce ESLint + Prettier standards in the TypeScript client
Applied to files:
clients/typescript/eslint.config.js
📚 Learning: 2025-10-23T13:26:37.198Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/general.mdc:0-0
Timestamp: 2025-10-23T13:26:37.198Z
Learning: Applies to clients/typescript/** : Update TypeScript client types and examples to match API changes
Applied to files:
clients/typescript/tests/contextforgeClient.test.tsclients/typescript/package.jsonclients/typescript/src/index.ts
📚 Learning: 2025-10-23T13:31:29.892Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/**/*.ts : Implement a consistent ApiError shape and a base ContextForgeError class for API errors
Applied to files:
clients/typescript/tests/contextforgeClient.test.ts
📚 Learning: 2025-10-23T13:31:29.892Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/**/*.{test,spec}.ts : Write tests using Jest/Vitest conventions; mock fetch; assert request shape and response handling
Applied to files:
clients/typescript/tests/contextforgeClient.test.tsclients/typescript/tests/setup.ts
📚 Learning: 2025-10-23T13:31:29.892Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/rollup.config.js : Use Rollup to build ESM, CJS, and minified UMD bundles with rollup/plugin-typescript (declarations off in bundling) and terser for minification
Applied to files:
clients/typescript/rollup.config.js
📚 Learning: 2025-10-23T13:31:29.892Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/tsconfig.json : tsconfig.json should enable strict type-checking and production-friendly options (e.g., strict, noImplicitAny, noUnusedLocals, declaration, sourceMap, outDir, rootDir, resolveJsonModule) and exclude tests from build
Applied to files:
clients/typescript/rollup.config.js
📚 Learning: 2025-10-23T13:26:37.198Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/general.mdc:0-0
Timestamp: 2025-10-23T13:26:37.198Z
Learning: Applies to clients/typescript/**/package.json : Bump TypeScript client version in package.json for breaking changes
Applied to files:
clients/typescript/package.json
🧬 Code graph analysis (1)
clients/typescript/tests/contextforgeClient.test.ts (1)
clients/typescript/src/index.ts (1)
ContextForgeClient(88-554)
🔇 Additional comments (10)
clients/typescript/package.json (2)
4-4: LGTM: ES module migration.The addition of
"type": "module"correctly enables ES module syntax throughout the TypeScript client, aligning with the migration of config files (jest.config.js, rollup.config.js) to ES module exports.
72-72: LGTM: Required for new ESLint config.The
globalspackage is correctly added as a devDependency to support the new ESLint flat config format ineslint.config.js.clients/typescript/rollup.config.js (1)
44-44: LGTM: More semantically correct option.Using
declarationDirinstead ofoutDiris the appropriate TypeScript compiler option for specifying where type declaration files should be emitted, asoutDiris intended for JavaScript output.clients/typescript/jest.config.js (1)
1-31: LGTM: Correct ES module migration.The change from
module.exportstoexport defaultis required now thatpackage.jsondeclares"type": "module". The configuration structure remains unchanged.clients/typescript/tests/setup.ts (1)
10-12: LGTM: Formatting-only change.The trailing comma addition is a style preference with no functional impact.
clients/typescript/src/index.ts (1)
120-120: LGTM: Formatting-only changes.All changes in this file are purely stylistic (quote style, parentheses) with no functional impact on the code behavior or API.
Also applies to: 261-261, 316-316, 319-319, 341-351, 364-367, 442-452, 472-476, 499-499, 538-545
clients/typescript/tests/contextforgeClient.test.ts (1)
1-280: LGTM: Formatting-only changes.All changes are stylistic adjustments (import formatting, trailing commas, line breaks) with no modifications to test logic, assertions, or behavior.
clients/typescript/eslint.config.js (3)
1-105: LGTM: Solid ESLint flat config migration.The new flat config correctly enforces strict TypeScript rules and aligns with coding guidelines. The structure properly separates source and test configurations.
58-58: tsconfig.test.json is properly configured and exists.The test configuration file is present and correctly set up with Jest and Node types, appropriate compiler options for testing, and proper include/exclude patterns.
101-102: Verify intentionality of broad .js/.mjs ignore pattern and add documentation.The ignore patterns confirmed to exclude three config files:
eslint.config.js,jest.config.js, androllup.config.js. While config file exclusion is often intentional in TypeScript projects, the lack of explanatory comments creates ambiguity. Additionally, the coding guidelines specify thatclients/typescript/**/*.{ts,tsx,js,jsx}should enforce ESLint standards.Recommended action: Verify whether this broad exclusion is intentional. If so, add a comment explaining the rationale (e.g., "Config files use CommonJS and have different linting requirements"). If config files should be linted, use a more precise ignore pattern that excludes build artifacts only (e.g.,
dist/**,node_modules/**,coverage/**) and move config files under TypeScript rules or create a separate linting config for them.
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (1)
clients/typescript/scripts/prepare.js(1 hunks)
🧰 Additional context used
📓 Path-based instructions (4)
clients/typescript/**
📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Update TypeScript client types and examples to match API changes
TypeScript client should use Husky with fast pre-commit checks and smoke-test pre-push workflow
Files:
clients/typescript/scripts/prepare.js
⚙️ CodeRabbit configuration file
clients/typescript/**: Ensure strict typing, accurate DTOs from OpenAPI, consistent error shapes, and robust timeout/retry semantics.
Prefer fetch/axios configurations with sane defaults; avoid throwing ambiguous any-typed errors.
Files:
clients/typescript/scripts/prepare.js
**/*
📄 CodeRabbit inference engine (.cursor/rules/security.mdc)
Avoid committing content matching common secret patterns (e.g., sk-..., AKIA..., ghp_..., password='...')
Files:
clients/typescript/scripts/prepare.js
clients/**/*
📄 CodeRabbit inference engine (CONTRIBUTING.md)
clients/**/*: Client libraries should follow sound API client design patterns
Client libraries must implement error handling and retry logic where appropriate
Ensure type safety and clear interfaces in client libraries
Provide documentation and examples for client libraries
Maintain cross-platform compatibility in client libraries
Files:
clients/typescript/scripts/prepare.js
clients/typescript/**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Enforce ESLint + Prettier standards in the TypeScript client
Files:
clients/typescript/scripts/prepare.js
🧠 Learnings (1)
📓 Common learnings
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: CONTRIBUTING.md:0-0
Timestamp: 2025-10-23T13:33:13.407Z
Learning: Applies to clients/typescript/**/*.{ts,tsx,js,jsx} : Enforce ESLint + Prettier standards in the TypeScript client
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/general.mdc:0-0
Timestamp: 2025-10-23T13:26:37.198Z
Learning: Applies to clients/typescript/** : Update TypeScript client types and examples to match API changes
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/rollup.config.js : Use Rollup to build ESM, CJS, and minified UMD bundles with rollup/plugin-typescript (declarations off in bundling) and terser for minification
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/general.mdc:0-0
Timestamp: 2025-10-23T13:26:37.198Z
Learning: Applies to clients/typescript/**/package.json : Bump TypeScript client version in package.json for breaking changes
🧬 Code graph analysis (1)
clients/typescript/scripts/prepare.js (1)
clients/typescript/scripts/dev.js (1)
path(21-21)
🔇 Additional comments (1)
clients/typescript/scripts/prepare.js (1)
8-13: LGTM! Correct ES module migration pattern.The migration from CommonJS to ES modules is implemented correctly. Using
fileURLToPath(import.meta.url)to derive__filenameandpath.dirname(__filename)to derive__dirnameis the standard approach for replicating CommonJS behavior in ES modules.
- Extract duplicated TypeScript rules into sharedTypeScriptRules constant - Replace inline rule objects in both base and test configs with shared constant - Maintains all original rules and configurations unchanged - Improves maintainability by eliminating code duplication - Follows DRY principle for better code organization
- Replace CommonJS require() calls with ES module import statements - Add __dirname and __filename polyfills using fileURLToPath and path.dirname - Import fileURLToPath from 'url' module for ES module compatibility - Maintains all original functionality while using modern ES module syntax - Consistent with prepare.js ES module conversion
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (1)
clients/typescript/eslint.config.js(1 hunks)
🧰 Additional context used
📓 Path-based instructions (4)
clients/typescript/**
📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Update TypeScript client types and examples to match API changes
TypeScript client should use Husky with fast pre-commit checks and smoke-test pre-push workflow
Files:
clients/typescript/eslint.config.js
⚙️ CodeRabbit configuration file
clients/typescript/**: Ensure strict typing, accurate DTOs from OpenAPI, consistent error shapes, and robust timeout/retry semantics.
Prefer fetch/axios configurations with sane defaults; avoid throwing ambiguous any-typed errors.
Files:
clients/typescript/eslint.config.js
**/*
📄 CodeRabbit inference engine (.cursor/rules/security.mdc)
Avoid committing content matching common secret patterns (e.g., sk-..., AKIA..., ghp_..., password='...')
Files:
clients/typescript/eslint.config.js
clients/**/*
📄 CodeRabbit inference engine (CONTRIBUTING.md)
clients/**/*: Client libraries should follow sound API client design patterns
Client libraries must implement error handling and retry logic where appropriate
Ensure type safety and clear interfaces in client libraries
Provide documentation and examples for client libraries
Maintain cross-platform compatibility in client libraries
Files:
clients/typescript/eslint.config.js
clients/typescript/**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Enforce ESLint + Prettier standards in the TypeScript client
Files:
clients/typescript/eslint.config.js
🧠 Learnings (3)
📓 Common learnings
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/rollup.config.js : Use Rollup to build ESM, CJS, and minified UMD bundles with rollup/plugin-typescript (declarations off in bundling) and terser for minification
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: CONTRIBUTING.md:0-0
Timestamp: 2025-10-23T13:33:13.407Z
Learning: Applies to clients/typescript/**/*.{ts,tsx,js,jsx} : Enforce ESLint + Prettier standards in the TypeScript client
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/package.json : package.json should expose proper entry points (main/module/types/exports) and include build/test/lint scripts (rollup, tsc, vitest, eslint)
📚 Learning: 2025-10-23T13:33:13.407Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: CONTRIBUTING.md:0-0
Timestamp: 2025-10-23T13:33:13.407Z
Learning: Applies to clients/typescript/**/*.{ts,tsx,js,jsx} : Enforce ESLint + Prettier standards in the TypeScript client
Applied to files:
clients/typescript/eslint.config.js
📚 Learning: 2025-10-23T13:31:29.892Z
Learnt from: CR
PR: VirtualAgentics/ConextForge_memory#0
File: .cursor/rules/typescript.mdc:0-0
Timestamp: 2025-10-23T13:31:29.892Z
Learning: Applies to clients/typescript/tsconfig.json : tsconfig.json should enable strict type-checking and production-friendly options (e.g., strict, noImplicitAny, noUnusedLocals, declaration, sourceMap, outDir, rootDir, resolveJsonModule) and exclude tests from build
Applied to files:
clients/typescript/eslint.config.js
🔇 Additional comments (3)
clients/typescript/eslint.config.js (3)
1-4: LGTM: Correct ESM imports for ESLint v9 flat config.The imports are properly structured for the flat config format with the correct packages for TypeScript linting.
6-24: LGTM: Shared rules extraction addresses previous feedback.The extraction of
sharedTypeScriptRuleseliminates duplication and enforces strict typing standards aligned with the coding guidelines. The rules appropriately enforce type safety withno-explicit-any, explicit return types, and strict boolean expressions.
83-91: Review comment is valid but requires developer confirmation.The script confirms that only configuration files (
eslint.config.js,jest.config.js,rollup.config.js) and utility scripts (scripts/dev.js,scripts/prepare.js) are being ignored—no source code JS files exist insrc/.The broad
*.jsand*.mjspatterns intentionally exclude these files from linting. This is defensible for config files (avoiding circular linting of the ESLint config itself), but utility scripts inscripts/may warrant linting depending on project standards.If these files should be linted, refactor the ignore patterns to be more specific:
ignores: [ 'dist/**', 'node_modules/**', 'coverage/**', - '*.js', - '*.mjs', ],Then selectively exclude only config files:
{ ignores: [ 'dist/**', 'node_modules/**', 'coverage/**', + 'eslint.config.js', + 'jest.config.js', + 'rollup.config.js', ], },
- Add eslint-config-prettier to devDependencies in package.json - Import eslint-config-prettier in ESLint configuration - Include prettier config in ESLint config array after js.configs.recommended - Disables Prettier-related ESLint rules to prevent conflicts - Ensures consistent code formatting between ESLint and Prettier - Maintains all existing ESLint rules while preventing style conflicts
- Extract duplicated global declarations into sharedGlobals constant - Include ...globals.node, ...globals.browser and readonly browser APIs - Replace base config globals with sharedGlobals reference - Replace test config globals with sharedGlobals + test-specific additions - Eliminates code duplication while maintaining all original functionality - Improves maintainability by centralizing global declarations
- Update package-lock.json to include eslint-config-prettier dependency - Resolves CI failure where npm ci couldn't install due to lock file mismatch - Ensures package.json and package-lock.json are in sync - Fixes TypeScript Client CI/CD workflow dependency installation
- Remove explicit fetch, setTimeout, clearTimeout declarations from sharedGlobals - These globals are already included in globals.browser - Eliminates duplication and improves maintainability
�� Repository Badges and Improvements
This PR adds comprehensive repository badges and improvements to enhance the professional appearance and discoverability of the ContextForge Memory repository.
✅ Changes Made
Badge Section Added
Documentation Created
CI/CD Integration
🎯 Benefits
🧪 Testing
This PR will test:
📋 Next Steps
After merge:
This enhances repository professionalism and discoverability while maintaining all existing functionality.
Summary by CodeRabbit
Chores
Chores / Linting
Style