Skip to content

fix(gbp): refresh profiles during sync#730

Merged
arberx merged 1 commit into
mainfrom
ax/fix-gbp-description-sync
Jun 22, 2026
Merged

fix(gbp): refresh profiles during sync#730
arberx merged 1 commit into
mainfrom
ax/fix-gbp-description-sync

Conversation

@arberx

@arberx arberx commented Jun 22, 2026

Copy link
Copy Markdown
Member

What

  • Refresh selected GBP location profile fields from the Business Information API at the start of gbp-sync.
  • Keep gbp_locations.syncedAt tied to the successful per-location sync transaction, so failed locations are not analyzed as covered by the run.
  • Add a regression test that verifies a stale empty description is replaced during sync.

Why

PR #729 added gbp-description-missing, but the signal read gbp_locations.description while gbp-sync only refreshed performance, place action, lodging, and Places data. A location whose owner description was fixed in Google could keep re-emitting a stale missing-description insight until the operator manually rediscovered locations.

Validation

  • pnpm exec vitest run --project canonry gbp-sync.test.ts intelligence-service-gbp.test.ts
  • pnpm --filter @ainyc/canonry typecheck
  • pnpm --filter @ainyc/canonry lint
  • Commit hook ran repo lint and passed with existing warnings.

@arberx arberx merged commit 5027185 into main Jun 22, 2026
11 checks passed
@arberx arberx deleted the ax/fix-gbp-description-sync branch June 22, 2026 00:05
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