Skip to content

Conversation

@ammar-agent
Copy link
Collaborator

Overview

Added comprehensive user-facing documentation for two key features:

  1. Project Secrets - Managing environment variables for projects
  2. Agentic Git Identity - Configuring separate Git identity for AI-generated commits

Changes

Project Secrets Documentation (docs/project-secrets.md)

  • Explains what project secrets are and common use cases
  • Step-by-step guide for managing secrets via the UI
  • Security considerations and best practices
  • Troubleshooting common issues

Agentic Git Identity Documentation (docs/agentic-git-identity.md)

  • Complete setup guide for using separate GitHub account for AI commits
  • Instructions for generating Classic GitHub tokens (simpler than fine-grained)
  • Git credential helper configuration to prevent keychain conflicts
  • Environment variable setup for author/committer identity
  • Comprehensive troubleshooting section

Testing

Both documents have been added to docs/SUMMARY.md for proper navigation.

Generated with cmux

- Created comprehensive guide for using separate git identity for AI commits
- Covers GitHub account setup, Classic token creation, and credential config
- Includes troubleshooting section for common issues
- Documents the credential helper chain fix for macOS keychain conflicts
- Added to docs/SUMMARY.md for navigation

_Generated with `cmux`_
- Created comprehensive guide for managing project secrets
- Covers secret management UI, security considerations, and troubleshooting
- Documents how secrets are injected as environment variables in bash commands
- Added to docs/SUMMARY.md for navigation

_Generated with `cmux`_
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Address Codex feedback (P1):
- Use 'gh auth setup-git' as recommended approach (scopes to GitHub only)
- Add warning about global credential helper replacement
- Provide both scoped and global replacement options
- Update troubleshooting to mention scoped configuration

This prevents breaking Git authentication for non-GitHub remotes (GitLab, Bitbucket, etc.)
while still properly configuring GitHub authentication via gh CLI.

_Generated with `cmux`_
@ammar-agent
Copy link
Collaborator Author

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. You're on a roll.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

ammar-agent and others added 2 commits October 8, 2025 13:24
- Merged macOS/Linux/Windows sections into single unified approach
- Now recommends using Project Secrets for GIT_AUTHOR_* and GIT_COMMITTER_* variables
- Updated troubleshooting sections to reference Project Secrets instead of shell config
- Updated reverting section to use Project Secrets UI
- Cross-referenced project-secrets.md documentation

This makes setup much simpler and keeps all project configuration in one place.
Preserves user's existing edits to intro section.

_Generated with `cmux`_
ammar-agent and others added 5 commits October 8, 2025 13:33
- Added 'Minimum Viable Documentation' style guide to README.md
- Emphasizes trusting users, removing fluff, and keeping only essential info
- Fixed icon references to use correct 🔑 (key) emoji instead of 🔒 (lock)

Based on recent doc simplification changes that cut ~200 lines of
unnecessary troubleshooting, step-by-step UI instructions, and
hypothetical best practices.

_Generated with `cmux`_
Instead of generic example, now prints exact commands for each PNG:
  cwebp 'docs/img/screenshot.png' -o 'docs/img/screenshot.webp' -q 85

_Generated with `cmux`_
@ammario ammario enabled auto-merge (squash) October 8, 2025 18:39
@ammario ammario disabled auto-merge October 8, 2025 18:55
@ammario ammario merged commit eddee9b into main Oct 8, 2025
7 of 8 checks passed
@ammario ammario deleted the sec-test branch October 8, 2025 18:55
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.

2 participants