Skip to content

docs: add portfolio context recovery note#114

Merged
saagpatel merged 1 commit into
masterfrom
codex/docs/context-recovery-dirty-batch-1
May 17, 2026
Merged

docs: add portfolio context recovery note#114
saagpatel merged 1 commit into
masterfrom
codex/docs/context-recovery-dirty-batch-1

Conversation

@saagpatel
Copy link
Copy Markdown
Owner

What

Adds a managed portfolio-context block to AGENTS.md.

Why

This closes one dirty-worktree item in the GithubRepoAuditor context recovery plan while leaving unrelated local dependency-lock work untouched.

How

Documentation-only update based on the README and repo-local agent guidance.

Checklist

  • Tests pass
  • No new warnings
  • Documentation updated (if applicable)

Testing

  • git diff --check
  • Placeholder scan for generic filler

@saagpatel saagpatel merged commit f38bf39 into master May 17, 2026
30 checks passed
@saagpatel saagpatel deleted the codex/docs/context-recovery-dirty-batch-1 branch May 17, 2026 04:01
Copy link
Copy Markdown

@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.

Reviewed commit: c6cd0c0e47

ℹ️ 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread AGENTS.md
- Lockfile rationale when lockfiles changed
- Screenshots for UI changes
6. Performance checks required before release health is considered done:
6. Performance checks required before done:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Preserve release scoping for performance gates

This makes the full performance suite mandatory before any task can be called done, and line 33 then blocks completion whenever one of those gates is not run. The previous wording scoped bundle/build/Lighthouse/API/DB/asset checks to release health, so a documentation-only or routine maintenance change now becomes impossible to complete without running release-level performance validation; keep these gates scoped to release validation or relevant performance-impacting changes.

Useful? React with 👍 / 👎.

Comment thread AGENTS.md
Comment on lines +69 to +71
## Current State

The repo is active product work. The README describes the intended support workflow, core search pipeline, and local privacy posture. Current local changes include unrelated dependency-lock work, so recovery edits should stay limited to documentation context unless the active branch owner explicitly broadens scope.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Remove stale worktree-specific guidance

This portfolio context is committed as repo-wide agent guidance, but it describes transient local state (Current local changes include unrelated dependency-lock work) and tells future agents to limit recovery edits around that state. On a clean branch or after the dependency-lock work is resolved, agents following AGENTS.md will still behave as if unrelated local changes exist and may incorrectly avoid legitimate non-documentation work; keep this section limited to stable project context or move ephemeral recovery notes outside committed instructions.

Useful? React with 👍 / 👎.

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