Ajan kataloğunda site kimliğini doğru sütunla ara

Katalog ucu canlıda boş dönüyordu. Sebep: `SiteGrantView.clientId` yabancı
anahtar, yani `oauth_clients.id`; ben onu metin `client_id` arayan
`getClientByClientId`'ye veriyordum. Sorgu hiçbir şey bulmuyor, null dönüyor
ve döngüdeki `continue` bunu "bu site eylem sunmuyor" diye yutuyordu — hata
hiçbir yerde görünmedi.

Asıl kusur yanlış çağrı değil, sessizliği. `getClientById` eklendi ve iki
çağrı düzeltildi.

Test ayrımı tutuyor: grant satır kimliğini taşımalı, satır kimliği metin
client_id olarak ARANAMAMALI. `getClientById`'yi yanlış sütuna bakacak şekilde
bozan mutasyon testi düşürüyor (24→23).

Bu hatayı birim testleri yakalayamazdı; yalnız canlıda uçtan uca deneyince
göründü ve o da ancak "200 döndü ama liste boş" diye bakınca.

Author: sametbasbug

scripts/orbit-d1-test-worker.ts

@@ -748,6 +748,12 @@ async function handleAction(body: ActionRequest, env: Environment): Promise<Resp
       return json({ ok: true });
     }
 
+    case 'getSiteClientById': {
+      return json({
+        client: await siteRepository.getClientById(stringValue(data, 'id')),
+      });
+    }
+
     case 'getSiteClient': {
       return json({ client: await siteRepository.getClientByClientId(stringValue(data, 'clientId')) });
     }

scripts/orbit-site-authorization-tests.ts

@@ -600,6 +600,37 @@ describe('Orbit as a sign-in door for other sites', { concurrency: false }, () =
     assert.equal(afterReplay.tokens.filter((row) => row.revoked_at === null).length, 0);
   });
 
+  test('a grant points at the client ROW id, not the public client_id', async () => {
+    const now = 1_760_000_950_000;
+    const accountId = 'door-rowid';
+    await seedClientAndAccount(accountId, now);
+    const grant = await consent({ prefix: 'rowid', accountId, now });
+
+    /* Bu test canlıda ödenen bir bedelden doğdu. `SiteGrantView.clientId`
+       yabancı anahtar, yani `oauth_clients.id`. Onu metin `client_id` arayan
+       `getClientByClientId`'ye verince sorgu hiçbir şey bulmuyor, null dönüyor
+       ve çağıran tarafta "bu site eylem sunmuyor" gibi görünüyor. Ajan
+       kataloğu boş döndü ve hiçbir yerde hata görünmedi — çünkü çağıran
+       taraftaki `continue` onu yutuyordu.
+
+       İki isim birbirine o kadar benziyor ki karıştırmak kaçınılmaz; bu yüzden
+       ayrımı test tutuyor. */
+    assert.equal(grant.clientId, CLIENT.id, 'grant satır kimliğini taşımalı');
+    assert.notEqual(grant.clientId, CLIENT.clientId, 'metin client_id ile aynı olmamalı');
+
+    const byRow = await callAction<{ client: { clientId: string } | null }>(
+      'getSiteClientById', { id: grant.clientId },
+    );
+    assert.equal(byRow.client?.clientId, CLIENT.clientId, 'satır kimliğiyle bulunmalı');
+
+    /* Ters yön: satır kimliğini metin arayan fonksiyona vermek BULMAMALI.
+       Bulsaydı hata görünür olurdu; bulmadığı için sessizdi. */
+    const byPublic = await callAction<{ client: unknown | null }>(
+      'getSiteClient', { clientId: grant.clientId },
+    );
+    assert.equal(byPublic.client, null, 'satır kimliği metin client_id olarak aranamaz');
+  });
+
   test('agent access is off until the person turns it on, and dies with the grant', async () => {
     const now = 1_760_000_900_000;
     const accountId = 'door-agent';

src/server/http/api.ts

@@ -7516,7 +7516,10 @@ export async function handleApiRequest(
       const sites = [];
       for (const grant of grants) {
         if (grant.revokedAt !== null || grant.agentAccessAt === null) continue;
-        const client = await siteRepository.getClientByClientId(grant.clientId);
+        /* `grant.clientId` SATIR kimliği (yabancı anahtar), metin client_id
+         * değil — `getClientByClientId` ile aramak hiçbir şey bulmuyordu ve
+         * site sessizce listeden düşüyordu. */
+        const client = await siteRepository.getClientById(grant.clientId);
         if (!client || client.status !== 'active' || !client.actionsUrl) continue;
         try {
           const { catalog, fetchedAt } = await siteActionCatalog(client, now);
@@ -7570,7 +7573,7 @@ export async function handleApiRequest(
         throw new ApiError(403, 'agent_access_closed', 'The person has not opened this site to agents.');
       }
 
-      const client = await siteRepository.getClientByClientId(grant.clientId);
+      const client = await siteRepository.getClientById(grant.clientId);
       if (!client || client.status !== 'active' || !client.actionsUrl || !client.actionsEndpoint) {
         throw new ApiError(503, 'site_catalog_unavailable', 'This site does not accept agent actions.');
       }

src/server/repositories/d1/d1-site-authorization-repository.ts

@@ -209,6 +209,25 @@ export class D1SiteAuthorizationRepository implements SiteAuthorizationRepositor
     return clientFromSql(row, uris.results.map((entry) => entry.redirect_uri));
   }
 
+  async getClientById(id: string): Promise<SiteClientView | null> {
+    const row = await this.#db.prepare(`
+      SELECT id, client_id, secret_digest, hash_version, label, site_url,
+             allowed_scopes, environment, status, created_at, revoked_at,
+             actions_url, actions_endpoint
+      FROM oauth_clients
+      WHERE id = ?
+    `).bind(id).first<ClientSqlRow>();
+    if (!row) return null;
+
+    const uris = await this.#db.prepare(`
+      SELECT redirect_uri FROM oauth_client_redirect_uris
+      WHERE client_id = ?
+      ORDER BY created_at, redirect_uri
+    `).bind(row.id).all<{ redirect_uri: string }>();
+
+    return clientFromSql(row, uris.results.map((entry) => entry.redirect_uri));
+  }
+
   async createClient(
     input: Parameters<SiteAuthorizationRepository['createClient']>[0],
   ): Promise<SiteClientView | null> {

src/server/repositories/site-authorization-repository.ts

@@ -91,6 +91,16 @@ export interface SiteTokenResolution {
 export interface SiteAuthorizationRepository {
   getClientByClientId(clientId: string): Promise<SiteClientView | null>;
 
+  /* SATIR KİMLİĞİYLE istemci. `getClientByClientId` metin `client_id`'yi arar
+   * (örn. "orbit-equinox-rota"); bu ise `oauth_clients.id`'yi.
+   *
+   * İkisi ayrı olmak zorunda ve karıştırmak sessiz: `SiteGrantView.clientId`
+   * yabancı anahtar, yani SATIR kimliği. Onu metin arayan fonksiyona verince
+   * sorgu hiçbir şey bulmuyor, `null` dönüyor ve çağıran tarafta "bu site
+   * eylem sunmuyor" gibi görünüyor. Canlıda tam olarak bu oldu: katalog boş
+   * döndü, hata da vermedi. */
+  getClientById(id: string): Promise<SiteClientView | null>;
+
   /* Yeni alt site istemcisi. Sır BURAYA GİRMİYOR, yalnız özeti: peper
    * Worker'ın içinde kalıyor ve özet çağıran tarafta hesaplanıyor.
    *