Skip to content

chore: open-source release cleanup#208

Merged
anandgupta42 merged 1 commit intomainfrom
chore/oss-cleanup
Mar 17, 2026
Merged

chore: open-source release cleanup#208
anandgupta42 merged 1 commit intomainfrom
chore/oss-cleanup

Conversation

@anandgupta42
Copy link
Contributor

What does this PR do?

Cleans up the repository for open-source release by removing internal files, fixing CI workflows, and standardizing branding.

Removed:

  • Internal planning docs (FEATURES_*.md, IMPLEMENTATION_PLAN.md, PLATFORM_ENGINEER_SIMULATION.md, STATS.md)
  • Internal scripts (beta.ts, duplicate-pr.ts, stats.ts)
  • Internal CI workflows (beta.yml, stats.yml)
  • .github/TEAM_MEMBERS (internal employee list)
  • .zed/ (single dev editor config)
  • docs/design/ (internal product strategy)

Fixed:

  • pr-management.yml: removed broken check-duplicates job (referenced deleted script), kept add-contributor-label
  • opencode.yml: replaced self-hosted blacksmith runner with ubuntu-latest
  • Standardized internal placeholder URLs from opencode.internalaltimate-code.internal

Moved:

  • RELEASING.mddocs/RELEASING.md

Type of change

  • Chore (maintenance, cleanup, no functional changes)

Issue for this PR

Closes #207

How did you verify your code works?

  • Verified all deleted files are internal-only (not referenced by source code or CI)
  • Confirmed pr-management.yml retains the useful add-contributor-label job
  • Grep confirmed all opencode.internal references updated to altimate-code.internal

Checklist

  • I have tested locally and verified my changes work
  • I have not included unrelated changes

@claude
Copy link

claude bot commented Mar 17, 2026

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review.

@anandgupta42 anandgupta42 force-pushed the chore/oss-cleanup branch 3 times, most recently from 6a2ba6b to 0e78386 Compare March 17, 2026 01:27
Copy link
Collaborator

@arora-saurabh448 arora-saurabh448 left a comment

Choose a reason for hiding this comment

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

  • Need to Remove or Update -> .github/CODEOWNERS (currently points to opencode users)

  • Need to remove or update -> .github/VOUCHED.td

  • Verify if the templates are required in .github folder

  • experiments/ folder needs to be removed or updated

  • github/ folder for Github actions needs to be made consistent - terminal commands still mention altimate-code instead of altimate standalone

  • Security.md needs to be updated to remove opencode contact email and put ours instead. It also mentions - Report a Vulnerability tab, which we do not have and the associated link is broken(404). Can either update on the basis of this or remove this section entirely.

  • LICENSE - says Copyright Opencode 2025. It should say both -

Copyright (c) 2026 Altimate AI
Copyright (c) 2026 opencode

Kilo also does the same - https://github.com/Kilo-Org/kilocode/blob/main/LICENSE

Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need this file? This has information about removal of files/content as logs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch! Removed it from tracking and added .github/meta/ to .gitignore. This was a scratch file used for commit messages — shouldn't be committed.

@anandgupta42 anandgupta42 force-pushed the chore/oss-cleanup branch 4 times, most recently from 2637146 to b2d410c Compare March 17, 2026 02:06
@anandgupta42
Copy link
Contributor Author

Addressed all review items:

  • CODEOWNERS — Keeping for now (will update with team ownership rules separately)
  • VOUCHED.td — Removed (upstream contributor vouching file, not relevant to our fork)
  • Templates — Verified: bug report, feature request, question, PR template all look good. Keeping as-is.
  • experiments/ — Keeping per team decision
  • github/ action naming — Already branded as altimate-code throughout. The CLI binary is altimate-code (with altimate as alias), so terminal commands are correct.
  • SECURITY.md — Fixed escalation email from security@anoma.lysecurity@altimate.ai. Enabled private vulnerability reporting on GitHub so the "Report a Vulnerability" tab works.
  • LICENSE — Added dual copyright:
    Copyright (c) 2026 Altimate AI
    Copyright (c) 2025 opencode
    

Also fixed:

  • .github/meta/commit.txt removed + gitignored
  • altimate_change marker mismatches in config.ts and skill.ts
  • Removed beta.yml branding test (file was deleted)

- Remove internal planning docs (FEATURES_*.md, IMPLEMENTATION_PLAN.md,
  PLATFORM_ENGINEER_SIMULATION.md, STATS.md)
- Remove internal scripts (beta.ts, duplicate-pr.ts, stats.ts)
- Remove internal CI workflows (beta.yml, stats.yml)
- Remove `.github/TEAM_MEMBERS` (internal employee list)
- Remove `.zed/` (single dev's editor config)
- Remove `docs/design/` (internal product strategy)
- Move `RELEASING.md` to `docs/`
- Fix `pr-management.yml`: remove broken `check-duplicates` job, keep
  `add-contributor-label`
- Fix `opencode.yml`: replace self-hosted runner with `ubuntu-latest`
- Standardize internal placeholder URLs to `altimate-code.internal`

Closes #207
@anandgupta42 anandgupta42 merged commit 3cf8656 into main Mar 17, 2026
7 checks passed
.describe(
"Automatically enhance prompts with AI before sending (default: false). Uses a small model to rewrite rough prompts into clearer versions.",
),
// altimate_change end
Copy link

Choose a reason for hiding this comment

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

Bug: An extra // altimate_change end comment in config.ts creates a mismatch with start markers, which will cause the CI merge guard test to fail.
Severity: HIGH

Suggested Fix

Remove the extraneous // altimate_change end comment from line 1269 in packages/opencode/src/config/config.ts to ensure the number of 'start' and 'end' markers is equal.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: packages/opencode/src/config/config.ts#L1269

Potential issue: An extra `// altimate_change end` comment was added in `config.ts`,
creating an imbalance with `// altimate_change start` comments. The file now contains 9
'start' markers but 10 'end' markers. This will cause the `upstream-merge-guard.test.ts`
CI check, which validates that these markers are balanced in all TypeScript source
files, to fail. The failure of this automated integrity check will prevent the pull
request from being merged.

Did we get this right? 👍 / 👎 to inform future reviews.

Comment on lines 13 to 19
startsWith(github.event.comment.body, '/altimate') ||
contains(github.event.comment.body, ' /ac') ||
startsWith(github.event.comment.body, '/ac')
runs-on: blacksmith-4vcpu-ubuntu-2404
runs-on: ubuntu-latest
permissions:
id-token: write
contents: read
Copy link

Choose a reason for hiding this comment

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

Bug: A script executed during the release CI workflow (release.yml) attempts to read the deleted .github/TEAM_MEMBERS file, which will cause the release process to fail.
Severity: CRITICAL

Suggested Fix

Remove the code from packages/script/src/index.ts that reads the now-deleted .github/TEAM_MEMBERS file to prevent the runtime error in the CI release workflow.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: .github/workflows/opencode.yml#L13-L19

Potential issue: The file `.github/TEAM_MEMBERS` was deleted, but the script at
`packages/script/src/index.ts` attempts to read it during module initialization. This
script is a dependency for other build and publish scripts (`build.ts`, `publish.ts`)
that are executed by the `release.yml` CI workflow. When the release workflow runs, it
will trigger the execution of these scripts, leading to a runtime error because the
required file is missing. This will cause the release pipeline to fail.

Did we get this right? 👍 / 👎 to inform future reviews.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: open-source release cleanup

2 participants