Skip to content

Feature flag tracker output#4438

Draft
jonathanKingston wants to merge 2 commits intomainfrom
jkt/auto/feature-flag-tracker-output-ed2a
Draft

Feature flag tracker output#4438
jonathanKingston wants to merge 2 commits intomainfrom
jkt/auto/feature-flag-tracker-output-ed2a

Conversation

@jonathanKingston
Copy link
Collaborator

@jonathanKingston jonathanKingston commented Feb 2, 2026

Asana Task/Github Issue:

Description

This PR updates the feature-flag-cleanup-tracker.yml workflow to operate on the constructed generated/v*/android-config.json output rather than directly on overrides/android-override.json.

Key changes:

  • Expanded triggers: The workflow now triggers on changes to features/**/*.json in addition to overrides/android-override.json.
  • Constructed output analysis: The workflow builds the full configuration for both the base and head commits and then compares the generated android-config.json files to detect new feature flags.
  • Updated Asana task instructions: The generated Asana task now correctly references the constructed output and guides users to modify the input files (features/ and overrides/) for cleanup.

This change provides a more robust and accurate tracking of new feature flags by analyzing the final configuration consumed by clients, regardless of whether the change originated from an override or a feature definition.


Open in Cursor Open in Web


Note

Medium Risk
Adds Node-based generation and dual checkouts to the workflow; failures or mismatched config versioning could prevent Asana cleanup tasks from being created even when flags are added.

Overview
Updates feature-flag-cleanup-tracker.yml to detect newly-added Android feature flags by comparing the generated generated/v*/android-config.json output between the PR base and head commits instead of reading overrides/android-override.json directly.

The workflow now also triggers on features/**/*.json, checks out both base/head into separate directories, runs npm ci + node index.js to build configs, derives the config version from CURRENT_CONFIG_VERSION, and updates Asana task text to reference the constructed file and instruct cleanup via features/ and/or overrides/android-override.json.

Written by Cursor Bugbot for commit ce97f42. This will update automatically on new commits. Configure here.

Co-authored-by: jkingston <jkingston@duckduckgo.com>
@cursor
Copy link

cursor bot commented Feb 2, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

github-actions bot pushed a commit that referenced this pull request Feb 2, 2026
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

👋 Don't forget to add an individual reviewer (in addition to those auto-added), as this will create a task for them in Asana.

👉 Please mark this as DRAFT unless there's an intention to merge this immediately.
👉 Click "Merge when ready" if you're happy for this to be automatically merged once reviewed. (If not available, ensure you've signed in to DuckDuckGo oauth.)
👉 Don't forget to add schema changes to validate if you're adding/changing a feature.

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Generated file outputs:

Time updated: Mon, 02 Feb 2026 13:18:58 GMT

legacy
29 files identical
  • trackers-unprotected-temporary.txt
  • v3/android-config.json
  • v3/extension-brave-config.json
  • v3/extension-bravemv3-config.json
  • v3/extension-chrome-config.json
  • v3/extension-chromemv3-config.json
  • v3/extension-config.json
  • v3/extension-edg-config.json
  • v3/extension-edge-config.json
  • v3/extension-edgmv3-config.json
  • v3/extension-firefox-config.json
  • v3/extension-safarimv3-config.json
  • v3/ios-config.json
  • v3/macos-config.json
  • v3/windows-config.json
  • v4/android-config.json
  • v4/extension-brave-config.json
  • v4/extension-bravemv3-config.json
  • v4/extension-chrome-config.json
  • v4/extension-chromemv3-config.json
  • v4/extension-config.json
  • v4/extension-edg-config.json
  • v4/extension-edge-config.json
  • v4/extension-edgmv3-config.json
  • v4/extension-firefox-config.json
  • v4/extension-safarimv3-config.json
  • v4/ios-config.json
  • v4/macos-config.json
  • v4/windows-config.json

⚠️ File is identical

latest
14 files identical
  • v5/android-config.json
  • v5/extension-brave-config.json
  • v5/extension-bravemv3-config.json
  • v5/extension-chrome-config.json
  • v5/extension-chromemv3-config.json
  • v5/extension-config.json
  • v5/extension-edg-config.json
  • v5/extension-edge-config.json
  • v5/extension-edgmv3-config.json
  • v5/extension-firefox-config.json
  • v5/extension-safarimv3-config.json
  • v5/ios-config.json
  • v5/macos-config.json
  • v5/windows-config.json

⚠️ File is identical

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

JSON approval analysis:

Time updated: Mon, 02 Feb 2026 13:29:03 GMT

latest

🎯 OVERALL APPROVAL STATUS

✅ AUTO-APPROVED

Co-authored-by: jkingston <jkingston@duckduckgo.com>
github-actions bot pushed a commit that referenced this pull request Feb 2, 2026
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.

2 participants