feat: infer organization country from location (CM-1366)

[skwowet]

Summary

• Infer organization country from location on write when country is missing, using the shared getCountry helper
• Dual-write to orgAttributes (source=system) and organizations.country, same pattern as location

Changes

• Add organizations.country column migration
• Register country as an org attribute with defaultColumn: 'country'
• Add system attribute source (priority after custom / lfx-serve)
• Infer country in prepareOrganizationData after attributes are resolved
• Cover findOrCreate (integrations, public API, members enrichment org upsert) and org enrichment apply path

[Copilot]

Pull request overview

Adds inferred organization countries across the shared DAL, enrichment workflow, and persistence layer.

Changes:

• Adds country fields and database migration.
• Infers country from location using a system attribute.
• Extends organization enrichment and legacy persistence paths.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
services/libs/types/src/organizations.ts	Adds organization country type.
services/libs/types/src/enums/organizations.ts	Adds the system attribute source.
services/libs/types/src/db/organizations.ts	Adds country to database rows.
services/libs/data-access-layer/src/organizations/utils.ts	Implements country inference.
services/libs/data-access-layer/src/organizations/types.ts	Extends DAL organization types.
services/libs/data-access-layer/src/organizations/base.ts	Persists and selects country.
services/libs/data-access-layer/src/organizations/attributesConfig.ts	Registers country and source priority.
services/apps/organizations_enrichment_worker/src/activities/enrichment.ts	Loads country and location during enrichment.
backend/src/database/repositories/organizationRepository.ts	Supports country in legacy updates.
backend/src/database/models/organization.ts	Maps the country column in Sequelize.
backend/src/database/migrations/V1786109400__add_country_to_organizations.sql	Adds the country column.

Suppressed comments (2)

services/libs/data-access-layer/src/organizations/utils.ts:223

• Once a system country has been inferred, this guard finds it in existingAttributes and permanently skips inference. If the default location later changes (for example, France to Germany), both the default country attribute and organizations.country remain stale. Treat only non-system country sources as authoritative, and refresh or clear the system-derived value when the default location changes.

  if (
    !attributes.some(({ name }) => name === 'country') &&
    !(existingAttributes ?? []).some(({ name }) => name === 'country')
  ) {

backend/src/database/migrations/V1786109400__add_country_to_organizations.sql:5

• Adding the PostgreSQL column does not make it available to Insights: services/libs/tinybird/datasources/organizations.datasource has no country field, and organizations_geo_distribution.pipe:27-34 still derives countries from location. As written, the stated Insights consumer cannot use this dual-write. Add the datasource field and update the relevant pipe to consume it (with any desired fallback).

  add column if not exists "country" text;

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Suppressed comments (1)

services/libs/data-access-layer/src/organizations/utils.ts:219

• This comment only restates the following block and conflicts with the repository rule that code should be self-explanatory rather than annotated with what it does. Please remove it.

  // Infer country from location when missing