Remove archive directory and clean up Streamlit references#170
Conversation
The archive directory contained two deprecated components: - mcp_client_integration: Old MCP client code no longer in use - vteam_shared_configs: Legacy configuration management system Removing to clean up the repository and reduce maintenance burden. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Removed all references to the deprecated Streamlit-based RFE builder system: - Updated .gitignore and .claudeignore to remove demos/rfe-builder paths - Replaced getting-started guide with Kubernetes-native deployment instructions - Updated lab exercises to reference current vTeam architecture - Changed developer guide to reflect Go backend, operator, and runner components - Fixed configuration examples to use ProjectSettings CRs instead of .streamlit/secrets.toml - Updated troubleshooting guides with kubectl/oc commands The documentation now accurately reflects the current Kubernetes-native architecture with Go backend, Kubernetes operator, and Claude Code runner components. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 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 👍.
Codex can also answer questions or update the PR. Try commenting
| name: default-settings | ||
| namespace: vteam-dev | ||
| spec: | ||
| apiKeys: | ||
| anthropic: "sk-ant-api03-your-key-here" |
There was a problem hiding this comment.
The quick-start snippet instructs users to create a ProjectSettings resource named default-settings with an apiKeys map, but the shipped CRD (components/manifests/crds/projectsettings-crd.yaml) only permits a singleton named projectsettings and defines spec.groupAccess/runnerSecretsName as its fields. Applying the YAML in the doc will be rejected by the cluster with validation errors and still leave users without a way to provide their Anthropic key. The example should reference a Kubernetes Secret via runnerSecretsName and use the required resource name.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
💡 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 👍.
Codex can also answer questions or update the PR. Try commenting
| name: default-settings | ||
| namespace: vteam-dev | ||
| spec: | ||
| apiKeys: | ||
| anthropic: "sk-ant-api03-your-key-here" |
There was a problem hiding this comment.
The quick-start snippet instructs users to create a ProjectSettings resource named default-settings with an apiKeys map, but the shipped CRD (components/manifests/crds/projectsettings-crd.yaml) only permits a singleton named projectsettings and defines spec.groupAccess/runnerSecretsName as its fields. Applying the YAML in the doc will be rejected by the cluster with validation errors and still leave users without a way to provide their Anthropic key. The example should reference a Kubernetes Secret via runnerSecretsName and use the required resource name.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
💡 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 👍.
Codex can also answer questions or update the PR. Try commenting
| name: default-settings | ||
| namespace: vteam-dev | ||
| spec: | ||
| apiKeys: | ||
| anthropic: "sk-ant-api03-your-key-here" |
There was a problem hiding this comment.
The quick-start snippet instructs users to create a ProjectSettings resource named default-settings with an apiKeys map, but the shipped CRD (components/manifests/crds/projectsettings-crd.yaml) only permits a singleton named projectsettings and defines spec.groupAccess/runnerSecretsName as its fields. Applying the YAML in the doc will be rejected by the cluster with validation errors and still leave users without a way to provide their Anthropic key. The example should reference a Kubernetes Secret via runnerSecretsName and use the required resource name.
Useful? React with 👍 / 👎.
💡 Codex Reviewhttps://github.com/ambient-code/vTeam/blob/373425e89be1cad07ffad995699696711a5037d7/docs/user-guide/getting-started.md#L75-L79 The quick-start snippet instructs users to create a ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting |
1 similar comment
💡 Codex Reviewhttps://github.com/ambient-code/vTeam/blob/373425e89be1cad07ffad995699696711a5037d7/docs/user-guide/getting-started.md#L75-L79 The quick-start snippet instructs users to create a ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting |
## Summary This PR removes obsolete code and documentation references to improve repository clarity and prevent confusion for new contributors. ## Changes ### 1. Archive Directory Removal - **Deleted 37 files** containing deprecated components: - `archive/mcp_client_integration/` - Old MCP client code no longer in use - `archive/vteam_shared_configs/` - Legacy configuration management system - **Removed 9,337 lines** of obsolete code ### 2. Documentation Cleanup Updated all documentation to reflect the current Kubernetes-native architecture: - **Configuration files:** - `.gitignore` - Removed `demos/rfe-builder/.streamlit/secrets.toml` entry - `.claudeignore` - Removed `demos/rfe-builder-archived/` entry - **User documentation:** - `docs/user-guide/getting-started.md` - Complete rewrite for Kubernetes deployment with OpenShift Local (CRC) - `docs/labs/index.md` - Updated setup instructions to use current architecture - `docs/labs/basic/lab-1-first-rfe.md` - Changed from Streamlit to vTeam web interface - **Developer documentation:** - `docs/developer-guide/index.md` - Updated tech stack: Go backend, Kubernetes operator, Claude Code runner - `docs/reference/index.md` - Changed config examples from `.streamlit/secrets.toml` to `ProjectSettings` CRs - `docs-plan.md` - Updated troubleshooting to use `kubectl`/`oc` commands ## Architecture Changes Reflected **Before:** Documentation referenced Streamlit-based RFE builder with LlamaDeploy **After:** Documentation accurately reflects: - Kubernetes Custom Resources (AgenticSession, ProjectSettings, RFEWorkflow) - Go backend REST API - Kubernetes operator for job orchestration - Python Claude Code runner in job pods - NextJS frontend ## Testing - Ran `markdownlint` on all modified documentation files - Verified no remaining references to `streamlit`, `rfe-builder`, or `demos/` paths - Confirmed documentation coherence and accuracy ## Impact - **Repository cleanup:** Reduces maintenance burden by removing 37 obsolete files - **Documentation accuracy:** New users will follow correct deployment instructions - **Developer onboarding:** Clear understanding of current architecture components 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude <noreply@anthropic.com>
2 participants
Summary
This PR removes obsolete code and documentation references to improve repository clarity and prevent confusion for new contributors.
Changes
1. Archive Directory Removal
archive/mcp_client_integration/- Old MCP client code no longer in usearchive/vteam_shared_configs/- Legacy configuration management system2. Documentation Cleanup
Updated all documentation to reflect the current Kubernetes-native architecture:
Configuration files:
.gitignore- Removeddemos/rfe-builder/.streamlit/secrets.tomlentry.claudeignore- Removeddemos/rfe-builder-archived/entryUser documentation:
docs/user-guide/getting-started.md- Complete rewrite for Kubernetes deployment with OpenShift Local (CRC)docs/labs/index.md- Updated setup instructions to use current architecturedocs/labs/basic/lab-1-first-rfe.md- Changed from Streamlit to vTeam web interfaceDeveloper documentation:
docs/developer-guide/index.md- Updated tech stack: Go backend, Kubernetes operator, Claude Code runnerdocs/reference/index.md- Changed config examples from.streamlit/secrets.tomltoProjectSettingsCRsdocs-plan.md- Updated troubleshooting to usekubectl/occommandsArchitecture Changes Reflected
Before: Documentation referenced Streamlit-based RFE builder with LlamaDeploy
After: Documentation accurately reflects:
Testing
markdownlinton all modified documentation filesstreamlit,rfe-builder, ordemos/pathsImpact
🤖 Generated with Claude Code