Skip to content

chore(cli): remove Cloudsmith package-repo migration notice - #401

Merged
victorvhs017 merged 4 commits into
mainfrom
chore/remove-cloudsmith-alerts
Sep 16, 2026
Merged

victorvhs017 merged 4 commits into
mainfrom
chore/remove-cloudsmith-alerts

Conversation

@victorvhs017

Copy link
Copy Markdown
Contributor

Description 📣

The Cloudsmith Linux package repo sunset is September 16, 2026. The CLI was still printing a daily migration notice (and README still had the banner) even though the cutover is done.

This branch drops that notice: no more DisplayPackageRepoMigrationNoticeWithWriter on command init, no migration-notice.json cache / 24h throttle, no INFISICAL_DISABLE_MIGRATION_NOTICE, and the README callout is gone. Update checks are unchanged.

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

Tests 🛠️

  1. Build the CLI and run any command (e.g. infisical --help or infisical secrets). Confirm stderr has no Cloudsmith / artifacts-cli.infisical.com migration banner.
  2. Confirm ~/.infisical/migration-notice.json is no longer created. An existing file can stay; it is unused.
  3. Confirm INFISICAL_DISABLE_MIGRATION_NOTICE has no effect (variable is unused).
  4. Confirm update-check still runs when not --silent, not in Docker, and not structured output.

…date CLI to no longer display migration notice for the Linux package repository transition from Cloudsmith.
@infisical-review-police

Copy link
Copy Markdown

💬 Discussion in Slack: #pr-review-cli-401-chore-cli-remove-cloudsmith-package-repo-migration-notice

Posted by Review Police — reviews, comments, new commits, and CI failures will stream into this channel.

@greptile-apps

greptile-apps Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge because the deprecated notice is removed cleanly and the update-check path remains intact.

Summary

This PR removes the completed Cloudsmith package-repository migration campaign while preserving the existing update-check flow.

  • Stops displaying the migration notice during command initialization.
  • Removes the notice implementation, cache handling, environment-variable override, and cache filename constant.
  • Removes the corresponding README banner.
  • Leaves update checks and their existing execution guards unchanged.

Reviews (1) · Last reviewed commit: "Remove migration notice from README and ..."

- Added `github.com/alessio/shellescape v1.4.1` as a new requirement.
- Updated `cloud.google.com/go/auth` from v0.18.1 to v0.18.2.
- Updated `golang.org/x/net` from v0.57.0 to v0.58.0.
- Updated `google.golang.org/genproto/googleapis/api` and `google.golang.org/genproto/googleapis/rpc` to newer versions.
- Updated `google.golang.org/grpc` from v1.82.1 to v1.83.2.
@linear

linear Bot commented Sep 16, 2026

Copy link
Copy Markdown

ENG-2487

- Updated `cloud.google.com/go/auth` to v0.18.2.
- Updated `golang.org/x/net` to v0.58.0.
- Updated `google.golang.org/genproto/googleapis/api` and `google.golang.org/genproto/googleapis/rpc` to newer versions.
- Updated `google.golang.org/grpc` to v1.83.2.
…ository

- Added important notice regarding the migration of the Linux package repository from Cloudsmith to `artifacts-cli.infisical.com`.
- Included instructions for users to update their artifact URL and access older versions.
@victorvhs017
victorvhs017 merged commit b0d52df into main Sep 16, 2026
20 checks passed
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