feat: Add feature system to Core

[google-labs-jules]

This commit introduces a new feature system to the Core struct. A new Features struct is defined in pkg/core/interfaces.go with an IsEnabled method to check if a feature is enabled. The Core struct is updated to include this new Features struct, which is initialized in the New function.

---

PR created automatically by Jules for task 10736067772562717918

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

Release Notes

• New Features
  • Feature flag support has been introduced to enable controlled availability of features within the system.

Walkthrough

Feature-flag functionality introduced to the Core type. A new Features type manages feature flags through a slice, with an IsEnabled method to verify flag status. The Core struct now includes a Features field, and Core initialisation ensures Features is instantiated.

Changes

Cohort / File(s)	Summary
Feature flags system pkg/core/interfaces.go	Introduced Features type with Flags slice and IsEnabled method; added Features field to Core struct
Core initialisation pkg/core/core.go	Updated Core instance creation to initialise Features field with empty Features struct

Poem

🐰 Flags now flutter in the Core's embrace,
Features dance with style and grace,
IsEnabled whispers, "Yes, I'm here,"
Control flows smooth, bright and clear,
A rabbit's logic, feature-fine! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "feat: Add feature system to Core" is directly aligned with the main objective of this changeset. The raw summary confirms that the primary changes are the introduction of a new Features struct with an IsEnabled method and the integration of this feature system into the Core struct. The title is concise, uses a clear commit convention format, and accurately summarizes the key change without being vague or misleading.
Description Check	✅ Passed	The pull request description is well-aligned with the actual changes documented in the raw summary. It accurately describes the introduction of the Features struct with an IsEnabled method, the addition of the Features field to the Core struct, and the initialization in the New function. The description is specific and informative, providing readers with a clear understanding of what was implemented without being overly vague or off-topic.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat-add-feature-system

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 111e5e1 and ac0df17.

📒 Files selected for processing (2)

• pkg/core/core.go (1 hunks)
• pkg/core/interfaces.go (1 hunks)

🔇 Additional comments (3)

pkg/core/core.go (1)

18-18: LGTM! Features initialisation is correct.

The Features field is properly initialised as an empty struct. The nil Flags slice will work correctly with the IsEnabled method, returning false for all features until flags are explicitly added.

pkg/core/interfaces.go (2)

41-41: LGTM! Features field properly integrated.

The Features field is correctly added to the Core struct and initialised in the New() constructor. This provides a clean API for feature flag functionality.

---

20-33: Code appears incomplete—IsEnabled() is defined but never used in the codebase.

The Features type is initialised empty and never populated or accessed. The IsEnabled() method has no call sites and no test coverage. Recommending a map-based refactoring is premature optimisation without evidence of:

• Flags being populated with realistic counts
• IsEnabled() being called frequently
• Concurrent access requirements

The thread-safety concern is valid IF Flags are mutated after initialisation, but no evidence of this pattern exists. Verify the intended usage of this feature flag system before optimising the implementation.

---

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

[Snider]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.