Skip to content

Feature/ci cd infrastructure#13

Merged
Pterjudin merged 10 commits intomainfrom
feature/ci-cd-infrastructure
Dec 19, 2025
Merged

Feature/ci cd infrastructure#13
Pterjudin merged 10 commits intomainfrom
feature/ci-cd-infrastructure

Conversation

@Pterjudin
Copy link

No description provided.

Tajudeen added 10 commits December 19, 2025 21:08
- Add GitHub Actions workflows (reusable workflows, PR checks, post-merge)
- Add Husky hooks (pre-commit, pre-push)
- Add lint-staged configuration
- Add E2E test setup with Playwright
- Add documentation (QUICK_START, REGRESSION_GUARD, REQUIRED_CHECKS)
- Add verification scripts
- Update package.json with E2E test scripts
- Add .d.ts declaration files for React component exports (mountVoidOnboarding, mountVoidSelectionHelper, mountVoidSettings, mountCtrlK, mountSidebar, mountVoidTooltip, mountVoidCommandBar)
- Fix null check bug in cortexideSelectionHelperWidget.ts (add null guard before _register)
- Fix diffLines call in findDiffs.ts (add explicit options parameter)
- Enable dts generation in tsup.config.js (manual declarations used due to memory constraints)

All 14 TypeScript errors resolved. Typecheck now passes.
No runtime changes - only type declarations added.
Remove CI/CD documentation files that are not needed for open source project:
- BRANCH_PROTECTION_SETUP.md
- PR_DESCRIPTION_TEMPLATE.md
- QUICK_START.md
- REGRESSION_GUARD_IMPLEMENTATION.md
- REQUIRED_CHECKS_POLICY.md
Remove references to @OpenCortexIDE/maintainers team that doesn't exist.
Restore CODEOWNERS to original state from main branch.
Remove all references to non-existent teams and users.
Leave minimal CODEOWNERS file for maintainer to update with their GitHub username.
Add @Pterjudin as code owner for all files.
- Fix missing 'id' in reusable lint workflow cache step
- Fix Playwright browser caching order (cache before install)
- Improve E2E test error handling with proper warning messages
- Add Windows 7z fallback to PowerShell compression tools
- Make prettier optional in lint-staged config
- Remove unnecessary warning from pre-push hook
- Improve post-merge packaging workflow with compile step

All workflows now have proper error handling and fallbacks.
- Fix E2E web server command to use ./scripts/code-web.sh with correct port
- Add clearer documentation for post-merge packaging implementation
- Update packaging workflow with TODO and implementation guidance

These changes address the recommended adjustments from PR review:
- E2E tests will now properly start the web server
- Packaging workflow has clear documentation for future implementation
- Add --max-old-space-size=4096 to tsup commands to prevent out-of-memory errors
- Apply memory limit to both build and watch modes
- Fixes ERR_WORKER_OUT_OF_MEMORY when building large React bundles

The build was failing with memory errors when creating large bundle files
(1.65 MB sidebar-tsx, 1.52 MB void-settings-tsx). This allocates 4GB of
heap memory to the tsup/esbuild process to handle these large bundles.
- Separate value export (diffLines) from type export (Change)
- Use 'export type' for Change to avoid declaration conflict
- Fixes compilation error: Export declaration conflicts with exported declaration of 'Change'

The issue was that tsup was generating both an interface declaration and
a type export for Change, causing a TypeScript compilation error.
@Pterjudin Pterjudin merged commit f112b37 into main Dec 19, 2025
4 checks passed
@Pterjudin Pterjudin deleted the feature/ci-cd-infrastructure branch December 19, 2025 22:21
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