Remove internal-only tooling from the external build (#352)
* Remove internal-only tooling without changing external runtime contracts This trims the lowest-risk internal-only surfaces first: deleted internal modules are replaced by build-time no-op stubs, the bundled stuck skill is removed, and the insights S3 upload path now stays local-only. The privacy verifier is expanded and the remaining bundled internal Slack/Artifactory strings are neutralized without broad repo-wide renames. Constraint: Keep the first PR deletion-heavy and avoid mass rewrites of USER_TYPE, tengu, or claude_code identifiers Rejected: One-shot DMCA cleanup branch | too much semantic risk for a first PR Confidence: medium Scope-risk: moderate Reversibility: clean Directive: Treat full-repo typecheck as a baseline issue on this upstream snapshot; do not claim this commit introduced the existing non-Phase-A errors without isolating them first Tested: bun run build Tested: bun run smoke Tested: bun run verify:privacy Not-tested: Full repo typecheck (currently fails on widespread pre-existing upstream errors outside this change set) * Keep minimal source shims so CI can import Phase A cleanup paths The first PR removed internal-only source files entirely, but CI provider and context tests import those modules directly from source rather than through the build-time no-telemetry stubs. This restores tiny no-op source shims so tests and local source imports resolve while preserving the same external runtime behavior. Constraint: GitHub Actions runs source-level tests in addition to bundled build/privacy checks Rejected: Revert the entire deletion pass | unnecessary once the import contract is satisfied by small shims Confidence: high Scope-risk: narrow Reversibility: clean Directive: For later cleanup phases, treat build-time stubs and source-test imports as separate compatibility surfaces Tested: bun run build Tested: bun run smoke Tested: bun run verify:privacy Tested: bun run test:provider Tested: bun run test:provider-recommendation Not-tested: Full repo typecheck (still noisy on this upstream snapshot) --------- Co-authored-by: anandh8x <test@example.com>
This commit is contained in:
@@ -25,11 +25,14 @@ import { getBinaryName, getPlatform } from './installer.js'
|
||||
const GCS_BUCKET_URL =
|
||||
'https://storage.googleapis.com/claude-code-dist-86c565f3-f756-42ad-8dfa-d59b1c096819/claude-code-releases'
|
||||
export const ARTIFACTORY_REGISTRY_URL =
|
||||
'https://artifactory.infra.ant.dev/artifactory/api/npm/npm-all/'
|
||||
process.env.CLAUDE_CODE_INTERNAL_ARTIFACTORY_REGISTRY_URL ?? ''
|
||||
|
||||
export async function getLatestVersionFromArtifactory(
|
||||
tag: string = 'latest',
|
||||
): Promise<string> {
|
||||
if (!ARTIFACTORY_REGISTRY_URL) {
|
||||
throw new Error('Internal artifactory registry URL is not configured')
|
||||
}
|
||||
const startTime = Date.now()
|
||||
const { stdout, code, stderr } = await execFileNoThrowWithCwd(
|
||||
'npm',
|
||||
@@ -152,6 +155,9 @@ export async function downloadVersionFromArtifactory(
|
||||
version: string,
|
||||
stagingPath: string,
|
||||
) {
|
||||
if (!ARTIFACTORY_REGISTRY_URL) {
|
||||
throw new Error('Internal artifactory registry URL is not configured')
|
||||
}
|
||||
const fs = getFsImplementation()
|
||||
|
||||
// If we get here, we own the lock and can delete a partial download
|
||||
|
||||
Reference in New Issue
Block a user