Remove custom API endpoints in favor of Payload's native REST API

- Removed custom endpoint handler and endpoints directory
- Removed enableApi configuration option from plugin
- Updated client hooks to use Payload's native collection API
- Updated documentation to reflect API changes
- Updated view component to handle Payload API response format

The plugin now uses Payload CMS's built-in REST API for the feature-flags collection, which provides standard query syntax, pagination, and automatic access control enforcement.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
2025-10-03 14:17:11 +02:00
parent 93673d1b14
commit a6712666af
11 changed files with 260 additions and 209 deletions

43
.github/workflows/pr-version-check.yml vendored Normal file
View File

@@ -0,0 +1,43 @@
name: PR Version Check
on:
pull_request:
branches:
- main
types: [opened, synchronize]
jobs:
version-check:
runs-on: ubuntu-latest
steps:
- name: Checkout PR branch
uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Get PR branch package.json version
id: pr-version
run: echo "version=$(node -p "require('./package.json').version")" >> $GITHUB_OUTPUT
- name: Get main branch package.json version
id: main-version
run: |
git checkout main
echo "version=$(node -p "require('./package.json').version")" >> $GITHUB_OUTPUT
- name: Compare versions
run: |
PR_VERSION="${{ steps.pr-version.outputs.version }}"
MAIN_VERSION="${{ steps.main-version.outputs.version }}"
echo "PR branch version: $PR_VERSION"
echo "Main branch version: $MAIN_VERSION"
if [ "$PR_VERSION" = "$MAIN_VERSION" ]; then
echo "❌ Version must be updated in package.json"
echo "Current version: $MAIN_VERSION"
echo "Please increment the version number before merging to main"
exit 1
else
echo "✅ Version has been updated from $MAIN_VERSION to $PR_VERSION"
fi

View File

@@ -0,0 +1,49 @@
name: Publish to NPM
on:
push:
branches:
- main
jobs:
publish:
runs-on: ubuntu-latest
permissions:
contents: write
steps:
- name: Checkout code
uses: actions/checkout@v4
- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: '20'
registry-url: 'https://registry.npmjs.org'
- name: Setup pnpm
uses: pnpm/action-setup@v4
- name: Install dependencies
run: pnpm install --frozen-lockfile
- name: Run tests
run: pnpm test
- name: Run build
run: pnpm build
- name: Get package version
id: package-version
run: echo "version=$(node -p "require('./package.json').version")" >> $GITHUB_OUTPUT
- name: Create and push git tag
run: |
git config user.name "GitHub Actions"
git config user.email "actions@github.com"
git tag -a "v${{ steps.package-version.outputs.version }}" -m "Release v${{ steps.package-version.outputs.version }}"
git push origin "v${{ steps.package-version.outputs.version }}"
- name: Publish to NPM
run: pnpm publish --access public --no-git-checks
env:
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

View File

@@ -56,10 +56,10 @@ The plugin follows Payload's plugin architecture with multiple exports:
- `getUserVariant()`: A/B testing variant selection - `getUserVariant()`: A/B testing variant selection
- `getFeatureFlagsByTag()`: Query flags by tags - `getFeatureFlagsByTag()`: Query flags by tags
#### API Endpoints (`src/endpoints/customEndpointHandler.ts`) #### API Access
- `GET /api/feature-flags` - List all active feature flags - The plugin uses Payload's native REST API for the feature-flags collection
- `GET /api/feature-flags/:flag` - Get specific flag data - Standard Payload query syntax is supported
- Only created when `enableApi: true` - Collection access controls are enforced
### Collection Schema ### Collection Schema
The plugin creates a feature flags collection with these key fields: The plugin creates a feature flags collection with these key fields:
@@ -93,9 +93,9 @@ The plugin integrates with Payload by:
4. Supporting full collection customization through `collectionOverrides` 4. Supporting full collection customization through `collectionOverrides`
### Security Considerations ### Security Considerations
- REST API endpoints are disabled by default (`enableApi: false`)
- Server-side hooks are the preferred method for accessing feature flags - Server-side hooks are the preferred method for accessing feature flags
- Collection access can be restricted through `collectionOverrides.access` - Collection access can be restricted through `collectionOverrides.access`
- API access follows standard Payload authentication and authorization
### Testing Setup ### Testing Setup
The development configuration (`dev/payload.config.ts`) includes: The development configuration (`dev/payload.config.ts`) includes:

View File

@@ -51,7 +51,6 @@ export default buildConfig({
defaultValue: false, // New flags start disabled defaultValue: false, // New flags start disabled
enableRollouts: true, // Allow percentage rollouts enableRollouts: true, // Allow percentage rollouts
enableVariants: true, // Allow A/B testing enableVariants: true, // Allow A/B testing
enableApi: false, // REST API endpoints
disabled: false, // Plugin enabled disabled: false, // Plugin enabled
}), }),
], ],
@@ -82,12 +81,6 @@ export type PayloadFeatureFlagsConfig = {
*/ */
enableVariants?: boolean enableVariants?: boolean
/**
* Enable REST API endpoints for feature flags
* @default false
*/
enableApi?: boolean
/** /**
* Disable the plugin while keeping the database schema intact * Disable the plugin while keeping the database schema intact
* @default false * @default false
@@ -166,10 +159,10 @@ payloadFeatureFlags({
### Security Considerations ### Security Considerations
**API Access Control:** When `enableApi: true`, the REST endpoints respect your collection access controls: **Collection Access Control:** The feature flags collection uses Payload's standard access control system:
```typescript ```typescript
// Example: Secure API access // Example: Secure collection access
access: { access: {
// Option 1: Simple authentication check // Option 1: Simple authentication check
read: ({ req: { user } }) => !!user, // Only authenticated users read: ({ req: { user } }) => !!user, // Only authenticated users
@@ -183,7 +176,7 @@ access: {
} }
``` ```
**Important:** The plugin does not implement separate API authentication - it uses Payload's collection access system for security. **Important:** The plugin uses Payload's native REST API for the collection, which respects all access control rules.
## Usage ## Usage
@@ -259,75 +252,109 @@ export default async function ProductPage({ userId }: { userId: string }) {
### Using Feature Flags via REST API ### Using Feature Flags via REST API
If you have `enableApi: true`, you can use the REST API endpoints: The plugin uses Payload's native REST API for the collection. You can access feature flags through the standard Payload REST endpoints:
```typescript ```typescript
// Check if a specific feature is enabled // Get all feature flags
const response = await fetch('/api/feature-flags/new-dashboard') const response = await fetch('/api/feature-flags')
const flag = await response.json() const result = await response.json()
// result.docs contains the array of feature flags
if (flag.enabled) { // Query specific feature flags
const response = await fetch('/api/feature-flags?where[name][equals]=new-dashboard')
const result = await response.json()
if (result.docs.length > 0 && result.docs[0].enabled) {
// Show new dashboard // Show new dashboard
} }
// Get all active feature flags // Get only enabled flags
const allFlags = await fetch('/api/feature-flags') const response = await fetch('/api/feature-flags?where[enabled][equals]=true')
const flags = await allFlags.json() const result = await response.json()
``` ```
**Important Security Notes:** **Important Security Notes:**
- REST API endpoints are disabled by default (`enableApi: false`)
- **API endpoints respect your collection access controls** - they don't bypass security - **API endpoints respect your collection access controls** - they don't bypass security
- Configure access permissions using `collectionOverrides.access` (see example above) - Configure access permissions using `collectionOverrides.access` (see example above)
- Anonymous users can only access flags if you explicitly allow it in access controls - Anonymous users can only access flags if you explicitly allow it in access controls
### API Endpoints ### API Endpoints
When `enableApi: true`, the plugin exposes the following endpoints: The plugin uses Payload's standard REST API endpoints for the feature-flags collection:
#### Get All Active Feature Flags #### Get All Feature Flags
```http ```http
GET /api/feature-flags GET /api/feature-flags
``` ```
Returns all enabled feature flags: Returns paginated feature flags:
```json ```json
{ {
"new-dashboard": { "docs": [
{
"id": "...",
"name": "new-dashboard",
"enabled": true, "enabled": true,
"rolloutPercentage": 50, "rolloutPercentage": 50,
"variants": null, "variants": null,
"metadata": {} "metadata": {},
"createdAt": "...",
"updatedAt": "..."
}, },
"beta-feature": { {
"id": "...",
"name": "beta-feature",
"enabled": true, "enabled": true,
"rolloutPercentage": 100, "rolloutPercentage": 100,
"variants": [ "variants": [
{ "name": "control", "weight": 50, "metadata": {} }, { "name": "control", "weight": 50, "metadata": {} },
{ "name": "variant-a", "weight": 50, "metadata": {} } { "name": "variant-a", "weight": 50, "metadata": {} }
], ],
"metadata": {} "metadata": {},
"createdAt": "...",
"updatedAt": "..."
} }
],
"totalDocs": 2,
"limit": 10,
"page": 1,
"totalPages": 1,
"pagingCounter": 1,
"hasPrevPage": false,
"hasNextPage": false
} }
``` ```
#### Get Specific Feature Flag #### Query Specific Feature Flag
```http ```http
GET /api/feature-flags/:flagName GET /api/feature-flags?where[name][equals]=new-dashboard
``` ```
Returns a specific feature flag: Returns matching feature flags:
```json ```json
{ {
"docs": [
{
"id": "...",
"name": "new-dashboard", "name": "new-dashboard",
"enabled": true, "enabled": true,
"rolloutPercentage": 50, "rolloutPercentage": 50,
"variants": null, "variants": null,
"metadata": {} "metadata": {},
"createdAt": "...",
"updatedAt": "..."
}
],
"totalDocs": 1,
"limit": 10,
"page": 1,
"totalPages": 1,
"pagingCounter": 1,
"hasPrevPage": false,
"hasNextPage": false
} }
``` ```

View File

@@ -40,7 +40,6 @@ pnpm dev
payloadFeatureFlags({ payloadFeatureFlags({
enableRollouts: true, // Percentage rollouts enableRollouts: true, // Percentage rollouts
enableVariants: true, // A/B testing enableVariants: true, // A/B testing
enableApi: true, // REST endpoints
defaultValue: false, // New flags start disabled defaultValue: false, // New flags start disabled
// + custom fields and permissions // + custom fields and permissions
}) })
@@ -48,12 +47,17 @@ payloadFeatureFlags({
## API Testing ## API Testing
The plugin uses Payload's native REST API:
```bash ```bash
# Get all flags # Get all flags
curl http://localhost:3000/api/feature-flags curl http://localhost:3000/api/feature-flags
# Get specific flag # Query specific flag
curl http://localhost:3000/api/feature-flags/new-feature curl 'http://localhost:3000/api/feature-flags?where[name][equals]=new-feature'
# Get only enabled flags
curl 'http://localhost:3000/api/feature-flags?where[enabled][equals]=true'
``` ```

View File

@@ -127,7 +127,6 @@ export default buildConfig({
// Enable all features // Enable all features
enableRollouts: true, enableRollouts: true,
enableVariants: true, enableVariants: true,
enableApi: true,
defaultValue: false, defaultValue: false,
// Custom collection configuration // Custom collection configuration

View File

@@ -1,6 +1,6 @@
{ {
"name": "@xtr-dev/payload-feature-flags", "name": "@xtr-dev/payload-feature-flags",
"version": "0.0.7", "version": "0.0.9",
"description": "Feature flags plugin for Payload CMS - manage feature toggles, A/B tests, and gradual rollouts", "description": "Feature flags plugin for Payload CMS - manage feature toggles, A/B tests, and gradual rollouts",
"license": "MIT", "license": "MIT",
"type": "module", "type": "module",

View File

@@ -1,78 +0,0 @@
import type { PayloadHandler } from 'payload'
export const customEndpointHandler = (collectionSlug: string): PayloadHandler =>
async (req) => {
const { payload } = req
const url = new URL(req.url || '')
const pathParts = url.pathname.split('/').filter(Boolean)
const flagName = pathParts[pathParts.length - 1]
// Check if we're fetching a specific flag
if (flagName && flagName !== 'feature-flags') {
try {
const result = await payload.find({
collection: collectionSlug,
where: {
name: {
equals: flagName,
},
},
limit: 1,
})
if (result.docs.length === 0) {
return Response.json(
{ error: 'Feature flag not found' },
{ status: 404 }
)
}
const flag = result.docs[0]
// Return simplified flag data
return Response.json({
name: flag.name,
enabled: flag.enabled,
rolloutPercentage: flag.rolloutPercentage,
variants: flag.variants,
metadata: flag.metadata,
})
} catch (error) {
return Response.json(
{ error: 'Failed to fetch feature flag' },
{ status: 500 }
)
}
}
// Fetch all feature flags
try {
const result = await payload.find({
collection: collectionSlug,
limit: 1000, // Adjust as needed
where: {
enabled: {
equals: true,
},
},
})
// Return simplified flag data
const flags = result.docs.reduce((acc: any, flag: any) => {
acc[flag.name] = {
enabled: flag.enabled,
rolloutPercentage: flag.rolloutPercentage,
variants: flag.variants,
metadata: flag.metadata,
}
return acc
}, {})
return Response.json(flags)
} catch (error) {
return Response.json(
{ error: 'Failed to fetch feature flags' },
{ status: 500 }
)
}
}

View File

@@ -17,14 +17,16 @@ export interface FeatureFlag {
/** /**
* Hook to fetch all active feature flags from the API * Hook to fetch all active feature flags from the API
*/ */
export function useFeatureFlags(): { export function useFeatureFlags(
flags: Record<string, FeatureFlag> | null initialFlags: Partial<FeatureFlag>[]
): {
flags: Partial<FeatureFlag>[]
loading: boolean loading: boolean
error: string | null error: string | null
refetch: () => Promise<void> refetch: () => Promise<void>
} { } {
const { config } = useConfig() const { config } = useConfig()
const [flags, setFlags] = useState<Record<string, FeatureFlag> | null>(null) const [flags, setFlags] = useState<Partial<FeatureFlag>[]>(initialFlags)
const [loading, setLoading] = useState(true) const [loading, setLoading] = useState(true)
const [error, setError] = useState<string | null>(null) const [error, setError] = useState<string | null>(null)
@@ -33,25 +35,49 @@ export function useFeatureFlags(): {
setLoading(true) setLoading(true)
setError(null) setError(null)
const response = await fetch(`${config.serverURL}${config.routes.api}/feature-flags`) // Use Payload's native collection API
const names = initialFlags.map(f => f.name).filter(Boolean)
const query = names.length > 0
? `?where[name][in]=${names.join(',')}&limit=1000`
: '?limit=1000'
const response = await fetch(`${config.serverURL}${config.routes.api}/feature-flags${query}`)
if (!response.ok) { if (!response.ok) {
if (response.status === 404) {
throw new Error('Feature flags API not enabled. Set enableApi: true in plugin config.')
}
throw new Error(`Failed to fetch feature flags: ${response.statusText}`) throw new Error(`Failed to fetch feature flags: ${response.statusText}`)
} }
const result = await response.json() const result = await response.json()
setFlags(result)
// Create a map of fetched flags by name for quick lookup
const fetchedFlagsMap = new Map<string, Partial<FeatureFlag>>()
if (result.docs && Array.isArray(result.docs)) {
result.docs.forEach((doc: any) => {
fetchedFlagsMap.set(doc.name, {
name: doc.name,
enabled: doc.enabled,
rolloutPercentage: doc.rolloutPercentage,
variants: doc.variants,
metadata: doc.metadata,
})
})
}
// Sort flags based on the order of names in initialFlags
const sortedFlags = initialFlags.map(initialFlag => {
const fetchedFlag = fetchedFlagsMap.get(initialFlag.name!)
// Use fetched flag if available, otherwise keep the initial flag
return fetchedFlag || initialFlag
})
setFlags(sortedFlags)
} catch (err) { } catch (err) {
const errorMessage = err instanceof Error ? err.message : 'Unknown error occurred' const errorMessage = err instanceof Error ? err.message : 'Unknown error occurred'
setError(errorMessage) setError(errorMessage)
setFlags(null)
} finally { } finally {
setLoading(false) setLoading(false)
} }
}, [config.serverURL, config.routes.api]) }, [config.serverURL, config.routes.api, initialFlags])
useEffect(() => { useEffect(() => {
void fetchFlags() void fetchFlags()
@@ -65,13 +91,13 @@ export function useFeatureFlags(): {
*/ */
export function useFeatureFlag(flagName: string): { export function useFeatureFlag(flagName: string): {
isEnabled: boolean isEnabled: boolean
flag: FeatureFlag | null flag: Partial<FeatureFlag> | null
loading: boolean loading: boolean
error: string | null error: string | null
} { } {
const { flags, loading, error } = useFeatureFlags() const { flags, loading, error } = useFeatureFlags([{ name: flagName }])
const flag = flags?.[flagName] || null const flag = flags.find(f => f.name === flagName) || null
const isEnabled = flag?.enabled || false const isEnabled = flag?.enabled || false
return { isEnabled, flag, loading, error } return { isEnabled, flag, loading, error }
@@ -96,19 +122,30 @@ export function useSpecificFeatureFlag(flagName: string): {
setLoading(true) setLoading(true)
setError(null) setError(null)
const response = await fetch(`${config.serverURL}${config.routes.api}/feature-flags/${flagName}`) // Use Payload's native collection API with query filter
const response = await fetch(
`${config.serverURL}${config.routes.api}/feature-flags?where[name][equals]=${flagName}&limit=1`
)
if (!response.ok) { if (!response.ok) {
if (response.status === 404) {
setFlag(null)
setError(`Feature flag '${flagName}' not found`)
return
}
throw new Error(`Failed to fetch feature flag: ${response.statusText}`) throw new Error(`Failed to fetch feature flag: ${response.statusText}`)
} }
const result = await response.json() const result = await response.json()
setFlag(result)
if (result.docs && result.docs.length > 0) {
const doc = result.docs[0]
setFlag({
name: doc.name,
enabled: doc.enabled,
rolloutPercentage: doc.rolloutPercentage,
variants: doc.variants,
metadata: doc.metadata,
})
} else {
setFlag(null)
setError(`Feature flag '${flagName}' not found`)
}
} catch (err) { } catch (err) {
const errorMessage = err instanceof Error ? err.message : 'Unknown error occurred' const errorMessage = err instanceof Error ? err.message : 'Unknown error occurred'
setError(errorMessage) setError(errorMessage)

View File

@@ -1,7 +1,5 @@
import type { Config, CollectionConfig, Field } from 'payload' import type { Config, CollectionConfig, Field } from 'payload'
import { customEndpointHandler } from './endpoints/customEndpointHandler.js'
export type CollectionOverrides = Partial< export type CollectionOverrides = Partial<
Omit<CollectionConfig, 'fields'> Omit<CollectionConfig, 'fields'>
> & { > & {
@@ -29,11 +27,6 @@ export type PayloadFeatureFlagsConfig = {
* @default true * @default true
*/ */
enableVariants?: boolean enableVariants?: boolean
/**
* Enable REST API endpoints for feature flags
* @default false
*/
enableApi?: boolean
/** /**
* Override collection configuration * Override collection configuration
*/ */
@@ -48,7 +41,6 @@ export const payloadFeatureFlags =
defaultValue = false, defaultValue = false,
enableRollouts = true, enableRollouts = true,
enableVariants = true, enableVariants = true,
enableApi = false,
collectionOverrides, collectionOverrides,
} = pluginOptions } = pluginOptions
@@ -207,23 +199,5 @@ export const payloadFeatureFlags =
path: '/feature-flags-overview', path: '/feature-flags-overview',
} }
// Add API endpoints if enabled
if (enableApi) {
// Add API endpoint for fetching feature flags
config.endpoints.push({
handler: customEndpointHandler(collectionSlug),
method: 'get',
path: '/feature-flags',
})
// Add endpoint for checking a specific feature flag
config.endpoints.push({
handler: customEndpointHandler(collectionSlug),
method: 'get',
path: '/feature-flags/:flag',
})
}
return config return config
} }

View File

@@ -53,17 +53,13 @@ const FeatureFlagsViewComponent = () => {
}) })
if (!response.ok) { if (!response.ok) {
if (response.status === 404) {
setError('Feature flags API not enabled. Set enableApi: true in plugin config.')
return
}
throw new Error(`Failed to fetch feature flags: ${response.statusText}`) throw new Error(`Failed to fetch feature flags: ${response.statusText}`)
} }
const result = await response.json() const result = await response.json()
// Convert the result object to an array if it's not already // Extract docs array from Payload API response
const flagsArray = Array.isArray(result) ? result : Object.values(result || {}) const flagsArray = result.docs || []
// Only update state if the component is still mounted (signal not aborted) // Only update state if the component is still mounted (signal not aborted)
if (!signal?.aborted) { if (!signal?.aborted) {