Skip to content

Conversation

@jsonbailey
Copy link
Contributor

Summary

This PR makes two changes to the SDK feature metadata:

  1. Renames the allFlags feature from "Getting all flags" to "Generate bootstrap details" (keeping the same allFlags ID)
  2. Adds three new storing data features for specific database backends:
    • storingDataConsul - Storing data (Consul)
    • storingDataDynamodb - Storing data (DynamoDB)
    • storingDataRedis - Storing data (Redis)

Both products/feature_info.json and schemas/sdk_metadata.sql are updated to keep them in sync.

Review & Testing Checklist for Human

  • JSON syntax validation: The diff shows an indentation change in feature_info.json where the new storingData* entries use 4-space indentation while the rest of the file uses 2-space. Verify the JSON is valid and consider fixing the indentation for consistency.
  • Verify the rename from "Getting all flags" to "Generate bootstrap details" is the intended change - this affects how the feature is displayed/referenced elsewhere
  • Confirm the new feature IDs (storingDataConsul, storingDataDynamodb, storingDataRedis) follow the expected naming conventions

Test Plan

  1. Validate JSON syntax: python3 -m json.tool products/feature_info.json
  2. Verify SQL syntax is correct for the new INSERT values

Notes

This change is part of updating SDK metadata to support more granular tracking of storing data features across different database backends.


Requested by: @jsonbailey
Link to Devin run: https://app.devin.ai/sessions/49b44fe704ac4382802b7b3dc07970a6

@jsonbailey jsonbailey requested a review from a team as a code owner January 16, 2026 20:22
@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@jsonbailey jsonbailey merged commit fec9934 into main Jan 16, 2026
4 checks passed
@jsonbailey jsonbailey deleted the devin/1768594870-add-storing-data-features branch January 16, 2026 20:46
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