Skip to content

Copilot #17

Merged
novatechflow merged 20 commits intomainfrom
copilot-chore
Feb 22, 2026
Merged

Copilot #17
novatechflow merged 20 commits intomainfrom
copilot-chore

Conversation

@novatechflow
Copy link
Member

Rebranding to Scalytics Copilot and Critical Bug Fixes

This PR handles the official rebranding of the project from "Scalytics Connect" to Scalytics Copilot and addresses several critical bugs discovered during the transition, including a broken migration runner.

Key Changes

1. Rebranding & Renaming

Updated all occurrences of "Scalytics Connect" to Scalytics Copilot across:
Backend configuration (package.json, swagger.js, etc.)
Frontend (manifest.json, index.html, and UI components)
Documentation (renamed several .md files in docs/ and updated content)
Environment setup scripts (setup-frontend.sh, start-app.sh)

2. Bug Fixes

Migration Runner Fix:
Resolved an issue in src/config/database.js where database migrations using the runMigration() entry point were being skipped. The runner now correctly supports both up() and runMigration() patterns.
This ensures the columns like isLoading in the messages table are properly created on fresh installs.

Inference Router Safety:
Added a null-check and default assignment for the parameters object in src/services/inferenceRouter.js to prevent TypeError when max_tokens deletion is attempted on an undefined object.

3. New Test Coverage

Inference Router Tests: Added test/inference-router.test.js to verify:

  • Correct routing to local vLLM.
  • Automatic fixing of non-alternating message sequences (critical for Gemma/Llama models).
  • Migration Integrity: Added test/migration-integrity.test.js to ensure all files in src/config/migrations export a valid entry point, preventing future "skipped migration" bugs.

Frontend Smoke Test:
Added frontend/src/App.test.js to verify the application renders without crashing.

Verification Results

Backend Tests

All backend tests passed successfully:
PASS test/migration-integrity.test.js
PASS test/inference-router.test.js
PASS test/error-utils.test.js
Test Suites: 3 passed, 3 total
Tests: 6 passed, 6 total

Frontend Tests

Verified the initial rendering logic:
PASS src/App.test.js
✓ renders learn react link (75 ms)

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@novatechflow novatechflow merged commit 013c2bf into main Feb 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant