Skip to content

ci: refresh service surface manifest for affiliate-marketing endpoint change#276

Merged
cvsz merged 1 commit into
mainfrom
codex/fix-outdated-service-surface-manifest
Apr 26, 2026
Merged

ci: refresh service surface manifest for affiliate-marketing endpoint change#276
cvsz merged 1 commit into
mainfrom
codex/fix-outdated-service-surface-manifest

Conversation

@cvsz
Copy link
Copy Markdown
Owner

@cvsz cvsz commented Apr 26, 2026

Motivation

  • The generated service surface manifest was out of sync after a new affiliate-marketing endpoint surfaced, causing CI's manifest drift check to fail.
  • The documentation snapshot must match the regenerated manifest so endpoint counts and write-policy counts remain accurate.

Description

  • Updated config/service-surface-manifest.json to include the new POST /v1/system/tiktok-shopee endpoint under the affiliate-marketing service and its corresponding write-policy entry.
  • Updated the generated snapshot docs/development/feature-impact-dive-2026-04-06.md to reflect the new endpoint count and write-policy count for affiliate-marketing.

Testing

  • Regenerated the manifest with python scripts/feature_impact_dive.py --write-manifest and confirmed the manifest file changed as expected.
  • Validated the manifest with python scripts/feature_impact_dive.py --validate-manifest, which completed successfully and resolves the previous drift check failure.

Codex Task

@cvsz cvsz merged commit 259d998 into main Apr 26, 2026
1 of 5 checks passed
@cvsz cvsz deleted the codex/fix-outdated-service-surface-manifest branch April 26, 2026 14:34
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a new endpoint, POST /v1/system/tiktok-shopee, to the affiliate-marketing service. The changes include updating the service manifest with the new endpoint and its security requirements, as well as updating the corresponding documentation for endpoint counts and security policies. I have no feedback to provide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant