Skip to content

✨ [FFL-2858] Feature Flags tab — catalog browsing (stacked PR 2 of 3) - #4916

Open
kellyw1806 wants to merge 1 commit into
kelly.wang/ffl-2597from
kelly.wang/ffl-2858
Open

✨ [FFL-2858] Feature Flags tab — catalog browsing (stacked PR 2 of 3)#4916
kellyw1806 wants to merge 1 commit into
kelly.wang/ffl-2597from
kelly.wang/ffl-2858

Conversation

@kellyw1806

@kellyw1806 kellyw1806 commented Jul 27, 2026

Copy link
Copy Markdown

Motivation

Stacked on #4913 (OAuth). With the sign in functionality, this PR helps include the fetch for the total feature-flag catalog. Users are now able to browse their feature-flag catalog. It does not involve overriding flags as that is the next PR (#4912).

Stack (review bottom-up)

PR Base What it adds
1 of 3 #4913 · FFL-2597 main OAuth sign-in
2 of 3 #4916 (this) · FFL-2858 ffl-2597 catalog browsing
3 of 3 #4912 · FFL-2596 ffl-2858 flag overrides

⚠️ Base is kelly.wang/ffl-2597, not main — merge #4913 first. This PR's diff is only the catalog layer.

Changes

  • Fetches the catalog via the OAuth-capable GET /api/ui/ffe/feature-flags endpoint (paginated), transparently refreshing the token (getValidAccessToken, added here).
  • Parses/maps flags + variants (defensive value parsing, dedupe-by-key), with search + type/tag filters and pagination.
  • Expands the flags tab to render the catalog.

Demo

This is what the three PRs look like working together:

Screen.Recording.2026-07-24.at.5.14.31.PM.mov

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change (oauth.spec.tsgetValidAccessToken; flagCatalog.spec.ts).
  • Added e2e/integration tests for this change.
  • Updated documentation and/or relevant AGENTS.md file

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7da Bot commented Jul 27, 2026

Copy link
Copy Markdown

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 179.09 KiB 179.09 KiB 0 B 0.00%
Rum Profiler 8.43 KiB 8.43 KiB 0 B 0.00%
Rum Recorder 21.12 KiB 21.12 KiB 0 B 0.00%
Logs 55.71 KiB 55.71 KiB 0 B 0.00%
Rum Salesforce N/A 137.07 KiB N/A N/A N/A
Rum Slim 137.06 KiB 137.06 KiB 0 B 0.00%
Worker 22.96 KiB 22.96 KiB 0 B 0.00%

@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Jul 27, 2026

Copy link
Copy Markdown

Pipelines  Tests

Unblock PR with BitsAI

⚠️ Warnings

🚦 3 Pipeline jobs failed

DataDog/browser-sdk | unit-bs: [safari-desktop]   View in Datadog   GitLab

DataDog/browser-sdk | e2e: [firefox-pinned]   View in Datadog   GitLab

DataDog/browser-sdk | e2e: [firefox]   View in Datadog   GitLab

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
Patch Coverage: 76.56%
Overall Coverage: 77.25% (+0.02%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 875de0b | Docs | Datadog PR Page | Give us feedback!

Comment thread developer-extension/src/panel/hooks/useSdkInfos.ts Outdated
Comment thread developer-extension/src/panel/hooks/useSdkInfos.ts
Comment thread developer-extension/package.json Outdated
Comment thread developer-extension/src/panel/components/tabs/flagsTab/oauth.spec.ts Outdated
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch 3 times, most recently from 84c903a to ba2f761 Compare July 27, 2026 19:19
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from 4d4f152 to 415b54b Compare July 27, 2026 19:28
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from ba2f761 to 5683248 Compare July 27, 2026 19:28
@kellyw1806
kellyw1806 marked this pull request as ready for review July 27, 2026 20:07
@kellyw1806
kellyw1806 requested a review from a team as a code owner July 27, 2026 20:07
chatgpt-codex-connector[bot]

This comment was marked as resolved.

@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from 415b54b to 4a0a996 Compare July 27, 2026 20:30
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 5683248 to 4af8227 Compare July 27, 2026 20:30
chatgpt-codex-connector[bot]

This comment was marked as resolved.

@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from 4a0a996 to de6b4ea Compare July 27, 2026 20:40
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 4af8227 to 7376fbf Compare July 27, 2026 20:40
chatgpt-codex-connector[bot]

This comment was marked as resolved.

@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from de6b4ea to 9b5a9f3 Compare July 27, 2026 20:54
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 7376fbf to fb4faa6 Compare July 27, 2026 20:54
chatgpt-codex-connector[bot]

This comment was marked as resolved.

@sbarrio
sbarrio requested a review from bdibon July 28, 2026 07:06
@BeltranBulbarellaDD

Copy link
Copy Markdown
Contributor

@kellyw1806 In my opinion I would keep the stacked PR 2 and 3 in draft until the first one is approved. That way after changes are requested in the first one, this one remains clean for the reviewer.

@kellyw1806
kellyw1806 marked this pull request as draft July 28, 2026 13:39
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from 9b5a9f3 to 6d5a341 Compare July 28, 2026 15:32
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from fb4faa6 to 230ae07 Compare July 28, 2026 15:32
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from 6d5a341 to 4596d8b Compare July 28, 2026 16:29
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 230ae07 to 673a2b1 Compare July 28, 2026 16:29
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from 4596d8b to a2d74a1 Compare July 28, 2026 17:19
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 673a2b1 to 2f033a5 Compare July 28, 2026 17:19
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from a2d74a1 to 59a5aa6 Compare July 28, 2026 17:39
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 2f033a5 to 98058ac Compare July 28, 2026 17:39
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from 59a5aa6 to dbf0fe9 Compare July 28, 2026 18:46
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch 2 times, most recently from e55866f to 322814a Compare July 28, 2026 20:01
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from dbf0fe9 to cf550d4 Compare July 28, 2026 21:30
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 322814a to 87a1c1a Compare July 28, 2026 21:30
Builds on FFL-2597: fetches the flag catalog via the OAuth-capable FFE UI
endpoint (transparently refreshing the token), and renders it with
search + type/tag filters and pagination. Read-only; overriding flags
comes in FFL-2596.
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2597 branch from cf550d4 to 3414896 Compare July 29, 2026 15:32
@kellyw1806
kellyw1806 force-pushed the kelly.wang/ffl-2858 branch from 87a1c1a to 875de0b Compare July 29, 2026 15:32
@kellyw1806
kellyw1806 marked this pull request as ready for review July 29, 2026 15:46
@kellyw1806
kellyw1806 requested review from BeltranBulbarellaDD and Copilot and removed request for Copilot July 29, 2026 15:59
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.

3 participants