Skip to content

Revert "docs: Update SDLC docs for token-gated approvals [doc-updater]" - #637

Merged
jwbron merged 1 commit into
mainfrom
revert-624-egg/doc-update-sdlc-tokens
Feb 13, 2026
Merged

Revert "docs: Update SDLC docs for token-gated approvals [doc-updater]"#637
jwbron merged 1 commit into
mainfrom
revert-624-egg/doc-update-sdlc-tokens

Conversation

@jwbron

@jwbron jwbron commented Feb 13, 2026

Copy link
Copy Markdown
Owner

Reverts #624 in favor of #635

@jwbron
jwbron merged commit a644cc3 into main Feb 13, 2026
3 checks passed

@egg-reviewer egg-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This PR is already merged. Reviewed the full diff — it's a clean documentation-only revert (0 additions, 67 deletions across 4 files) of PR #624.

No blocking issues.

One observation: This reverts the documentation for token-gated approvals but the underlying implementation code from PR #616 (API endpoints, hook scripts, token generation) presumably still exists in the codebase. This creates a docs-code drift where the feature is present but undocumented. If the intent per issue #635 is to replace this approach entirely, consider also removing or deprecating the implementation code in a follow-up.

— Authored by egg

@egg-reviewer egg-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No agent-mode design concerns. This is a clean documentation revert removing token-gated approval docs in favor of #635. No agent workflows, pre-fetching, structured output requirements, or prompt-level constraints are affected.

— Authored by egg

@james-in-a-box

This comment has been minimized.

@james-in-a-box

Copy link
Copy Markdown
Contributor

egg agent-mode-design completed. View run logs

1 previous review(s) hidden.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant