Skip to content

[Fix] [CI/Server]: build ats-engine in workflows and configure database url for lint - #200

Merged
Gautam25Raj merged 2 commits into
masterfrom
fix/ci-build-shared-packages-and-lint-format
Sep 13, 2026
Merged

Gautam25Raj merged 2 commits into
masterfrom
fix/ci-build-shared-packages-and-lint-format

Conversation

@Gautam25Raj

Copy link
Copy Markdown
Collaborator

Description

During CI runs on clean environments, three workflow failures occurred:

  1. @veriworkly/server failed TypeScript compilation and Vitest suites because @veriworkly/ats-engine was not compiled in the Build Shared Packages step (only @veriworkly/profile-core was listed).
  2. The apps/server prebuild script only executed Prisma generation without compiling its workspace dependencies, which caused compilation to fail when dist was absent.
  3. The Lint & Format workflow failed during npm ci because Prisma 7 requires DATABASE_URL during postinstall (npm run db:generate).

This PR addresses all three issues:

  • Adds @veriworkly/ats-engine to the Build Shared Packages step in both .github/workflows/builds.yaml and .github/workflows/tests.yaml.
  • Updates apps/server/package.json to compile @veriworkly/profile-core and @veriworkly/ats-engine during prebuild.
  • Adds DATABASE_URL: "postgresql://dummy:dummy@localhost:5432/dummy" to .github/workflows/lint-format.yaml so npm ci finishes without errors.

Linked Issue

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🧪 Tests (adding or updating tests)

Checklist

  • I have read the CONTRIBUTING.md document.
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • Any dependent changes have been merged and published in downstream modules.

@Gautam25Raj Gautam25Raj added bug Something isn't working backend server-side logic dependencies Pull requests that update a dependency file labels Sep 13, 2026
@Gautam25Raj Gautam25Raj self-assigned this Sep 13, 2026
@Gautam25Raj
Gautam25Raj merged commit 9d65b21 into master Sep 13, 2026
11 checks passed
@Gautam25Raj
Gautam25Raj deleted the fix/ci-build-shared-packages-and-lint-format branch September 13, 2026 11:44
@Gautam25Raj Gautam25Raj changed the title fix(ci): build ats-engine in workflows and configure database url for lint [Fix] [CI/Server]: build ats-engine in workflows and configure database url for lint Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend server-side logic bug Something isn't working dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant