feat: Base data structures fot hook implementation.

[NeaguGeorgiana23]

This PR

Introduces HookHints and FlagTypeValus that will allow later implementation of Hooks.

Related Issues

Fixes #111

Follow-up Tasks

Implementing the EvaluationOptions and HookData.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: aa0908c9-2a46-4352-b842-645d22bb6243

📥 Commits

Reviewing files that changed from the base of the PR and between 5cc878b and 0e70a89.

📒 Files selected for processing (1)

• openfeature/flag_type_value.h

🚧 Files skipped from review as they are similar to previous changes (1)

• openfeature/flag_type_value.h

---

📝 Walkthrough

Walkthrough

This PR adds the FlagValueType enum class and HookHints type alias to the OpenFeature C++ SDK, along with header-only Bazel targets exposing both new headers.

Changes

New Type Definitions

Layer / File(s)	Summary
FlagValueType enum openfeature/flag_type_value.h, openfeature/BUILD	Defines openfeature::FlagValueType with boolean, string, integer, double, and object variants, and adds its header-only Bazel target.
HookHints type alias openfeature/hook_hints.h, openfeature/BUILD	Defines openfeature::HookHints as an std::unordered_map<std::string, std::any> alias and adds its header-only Bazel target.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• open-feature/cpp-sdk#118: Adds overlapping FlagValueType and HookHints headers and corresponding Bazel targets.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	The PR only adds HookHints and FlagValueType, but issue #111 also requires evaluation options, hook data, and hook context.	Add the remaining hook data structures or clearly document that this PR only partially closes #111.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is concise and relevant to the new hook-related base data structures.
Description check	✅ Passed	The description matches the changes and mentions the hook-related follow-up work.
Out of Scope Changes check	✅ Passed	The new headers and BUILD targets are all related to the hook-implementation groundwork.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Pull request overview

Adds foundational public types to support a future Hooks implementation in the OpenFeature C++ SDK, aligning with Issue #111’s “Hook hints” portion and preparing for later HookData/EvaluationOptions work.

Changes:

• Introduces HookHints as a std::unordered_map<std::string, std::any> type alias.
• Introduces FlagValueType enum to represent supported flag value categories.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
openfeature/hook_hints.h	Adds the HookHints type alias for passing arbitrary hook hint data.
openfeature/flag_type_value.h	Adds the FlagValueType enum for flag value categorization.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.