chore: workflow cleanup (#48)

Signed-off-by: Jeremy Andrews <[email protected]>

Author: ABC2015

.github/workflows/contributor-workflow.yml

@@ -1,5 +1,4 @@
-# This workflow analyzes the Dart Server SDK and a Go-feature-integration example project
-# to identify issues and commits for resolution.
+# This workflow analyzes the Dart Server SDK and an example project
 
 name: Contributor Workflow
 

.github/workflows/parent-pipeline.yml

@@ -60,7 +60,13 @@ jobs:
 #    with:
 #      branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
 #      event_name: ${{ needs.determine-workflow.outputs.event_name }}
-
+#  trigger-development-workflow:
+#    name: Trigger Development Workflow
+#    needs: trigger-validation
+#    if: |
+#      github.event_name == 'pull_request' &&
+#      github.base_ref == 'development'
+#    uses: ./.github/workflows/development-workflow.yml
 #  trigger-beta-workflow:
 #    name: Trigger Beta Workflow
 #    needs: trigger-validation

.github/workflows/release.yaml

@@ -33,4 +33,21 @@ jobs:
       release_tag_name: ${{ steps.release.outputs.tag_name }}
       upload_url: ${{ steps.release.outputs.upload_url }}
 
+# Publishing is handled in publish.yaml
+  dart-release:
+    needs: release-please
+    if: ${{ fromJSON(needs.release-please.outputs.release_created || false) }}
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      id-token: write # Required for authentication using OIDC
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - uses: dart-lang/setup-dart@v1
+      - name: Install dependencies
+        run: dart pub get
+      - name: Publish
+        run: dart pub publish --force
+
 # Publishing is handled in publish.yaml

.github/workflows/validation-workflow.yml

@@ -1,4 +1,4 @@
-# This workflow valides that branch and commit names are standardized for better workflow.
+# This workflow valides that branch and commit names are standardized.
 
 name: Validation Workflow
 
@@ -24,13 +24,13 @@ jobs:
           BRANCH_NAME="${{ inputs.branch_name }}"
           echo "Branch name: $BRANCH_NAME"
 
-          if [[ "$BRANCH_NAME" =~ ^(qa|development|main)$ ]]; then
+          if [[ "$BRANCH_NAME" =~ ^(qa|development|main|release-please--branches--main--components--openfeature_dart_server_sdk)$ ]]; then
             echo "✅ Branch name '$BRANCH_NAME' is valid for protected branches (qa, development, main)."
           elif [[ "$BRANCH_NAME" =~ ^(feat|fix|hotfix|chore|test|refactor|release)/[a-z0-9_-]+$ ]]; then
             echo "✅ Branch name '$BRANCH_NAME' follows the naming convention."
           else
             echo "❌ Branch name '$BRANCH_NAME' does not follow the naming convention: <type>/<branch-name>"
-            echo "Valid types: feat, fix, hotfix, chore, test, refactor, release, development, qa, main"
+            echo "Valid types: feat, fix, hotfix, chore, test, refactor, release, qa, main, development"
             exit 1
           fi
 

.gitignore

@@ -1 +1,2 @@
-.dart_tool/
+.dart_tool/
+pubspec.lock

CONTRIBUTING.md

@@ -2,14 +2,11 @@
 
 Thank you for contributing to this project. We value your input, and any issues or pull requests adhering to these guidelines are welcome.
 
-
-
 ## **Code of Conduct**
 
 Please read and follow our [Code of Conduct](https://github.com/open-feature/.github/blob/main/CODE_OF_CONDUCT.md).  
 **TL;DR**: Be respectful and professional.
 
-
 ## **Vendor Specific Details**
 
 Vendor specific details are intentionally not included in this module in order to be lightweight and agnostic.
@@ -168,7 +165,9 @@ We follow [Conventional Commits](https://www.conventionalcommits.org) to ensure
 To maintain consistency and ensure stability, we enforce the following **branch protection rules**:
 
 #### **Protected Branches**
-- **Branches**: `main`, `qa`, `beta`
+
+- **Branches**: `main`, `qa`, `development`
+
 - **Rules**:
   - Direct **pushes** are not allowed.
   - Changes must go through a **pull request** and pass all required checks before merging.
@@ -185,7 +184,7 @@ To maintain consistency and ensure stability, we enforce the following **branch
 #### **Branch Lifecycle**
 - **Feature, Fix, Hotfix, Test Branches**:
   - Created by developers for specific tasks.
-  - Merged into `main`, `qa`, or `beta` branches through pull requests.
+  - Merged into `main`, `qa`, or `development` branches through pull requests.
   - Deleted after merging.
 
 ---
@@ -197,7 +196,8 @@ To maintain consistency and ensure stability, we enforce the following **branch
    - Pushes to these branches are allowed without restrictions.
 
 2. **Pull Requests into Protected Branches**:
-   - Protected branches (`main`, `qa`, `beta`) require pull requests.
+
+   - Protected branches (`main`, `qa`, `development`) require pull requests.
    - Pull requests trigger workflows for testing and validation.
 
 3. **Validation on Push and Pull Requests**:

README.md

@@ -18,6 +18,7 @@
   <!-- x-release-please-start-version -->
   <a href="https://github.com/open-feature/dart-server-sdk/releases/tag/v0.0.7">
     <img alt="Release" src="https://img.shields.io/static/v1?label=release&message=v0.0.7&color=blue&style=for-the-badge" />
+
   </a>
   <a href="https://dart.dev/">
     <img alt="Built with Dart" src="https://img.shields.io/badge/Built%20with-Dart-blue.svg?style=for-the-badge" />
@@ -87,8 +88,6 @@ If the provider you're looking for hasn't been created yet, see the [develop a p
 
 Once you've added a provider as a dependency, it can be registered with OpenFeature like this:
 
-
-
 ### Targeting
 
 Sometimes, the value of a flag must consider some dynamic criteria about the application or user, such as the user's location, IP, email address, or the server's location.

lib/feature_provider.dart

@@ -11,7 +11,7 @@ enum ProviderState {
   DEGRADED,
   RECONNECTING,
   PLUGIN_ERROR,
-  MAINTENANCE
+  MAINTENANCE,
 }
 
 /// Provider configuration
@@ -118,7 +118,7 @@ class InMemoryProvider implements FeatureProvider {
   final ProviderConfig _config;
 
   InMemoryProvider(this._flags, [ProviderConfig? config])
-      : _config = config ?? const ProviderConfig();
+    : _config = config ?? const ProviderConfig();
 
   @override
   String get name => 'InMemoryProvider';
@@ -257,8 +257,4 @@ abstract class CommercialProvider implements FeatureProvider {
   ProviderState get state => _state;
 
   // HTTP request implementation template
-  Future<dynamic> _makeRequest(String path,
-      {Map<String, dynamic>? params}) async {
-    throw UnimplementedError('_makeRequest must be implemented by child class');
-  }
 }

lib/transaction_context.dart

@@ -35,13 +35,12 @@ class TransactionContext {
 /// Transaction context manager
 class TransactionContextManager {
   static final TransactionContextManager _instance =
-      TransactionContextManager._internal();
+      TransactionContextManager._internal(Duration(minutes: 5));
   final _contexts = <String, TransactionContext>{};
   final _contextStack = <String>[];
   final Duration _defaultTimeout;
 
-  TransactionContextManager._internal(
-      [this._defaultTimeout = const Duration(minutes: 30)]);
+  TransactionContextManager._internal(this._defaultTimeout);
 
   factory TransactionContextManager() => _instance;
 

local_dev_tools/validate_branch.dart

@@ -14,7 +14,7 @@ void main(List<String> args) {
 
   print('Branch name: $branchName');
 
-  final validBranches = RegExp(r'^(qa|beta|main)$');
+  final validBranches = RegExp(r'^(qa|development|main)$');
   final validFeatureBranch = RegExp(
     r'^(feat|fix|hotfix|chore|test|refactor|release)/[a-z0-9_-]+$',
   );
@@ -27,7 +27,7 @@ void main(List<String> args) {
       '❌ Branch name does not follow the required convention: <type>/<branch-name>',
     );
     print(
-      'Valid types: feat, fix, hotfix, chore, test, refactor, release, development, qa, main',
+      'Valid types: feat, fix, hotfix, chore, test, refactor, release, qa, development, main',
     );
     exit(1);
   }