Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
116 changes: 105 additions & 11 deletions cmd/version.go
Original file line number Diff line number Diff line change
@@ -1,17 +1,75 @@
package cmd

import (
"context"
"encoding/json"
"fmt"
"log"
"maps"
"slices"
"time"

"github.com/spf13/afero"
"github.com/terraform-linters/tflint/plugin"
"github.com/terraform-linters/tflint/tflint"
"github.com/terraform-linters/tflint/versioncheck"
)

const (
versionCheckTimeout = 3 * time.Second
)

// VersionOutput is the JSON output structure for version command
type VersionOutput struct {
Version string `json:"version"`
Plugins []PluginVersion `json:"plugins"`
UpdateCheckEnabled bool `json:"update_check_enabled"`
UpdateAvailable bool `json:"update_available"`
LatestVersion string `json:"latest_version,omitempty"`
}

// PluginVersion represents a plugin's name and version
type PluginVersion struct {
Name string `json:"name"`
Version string `json:"version"`
}

func (cli *CLI) printVersion(opts Options) int {
// For JSON format: perform synchronous version check
if opts.Format == "json" {
var updateInfo *versioncheck.UpdateInfo
if versioncheck.Enabled() {
ctx, cancel := context.WithTimeout(context.Background(), versionCheckTimeout)
defer cancel()

info, err := versioncheck.CheckForUpdate(ctx, tflint.Version)
if err != nil {
log.Printf("[ERROR] Failed to check for updates: %s", err)
} else {
updateInfo = info
}
}
return cli.printVersionJSON(opts, updateInfo)
}

// For text format: start async version check
var updateChan chan *versioncheck.UpdateInfo
if versioncheck.Enabled() {
updateChan = make(chan *versioncheck.UpdateInfo, 1)
go func() {
ctx, cancel := context.WithTimeout(context.Background(), versionCheckTimeout)
defer cancel()

info, err := versioncheck.CheckForUpdate(ctx, tflint.Version)
if err != nil {
log.Printf("[ERROR] Failed to check for updates: %s", err)
}
updateChan <- info
close(updateChan)
}()
}

// Print version immediately
fmt.Fprintf(cli.outStream, "TFLint version %s\n", tflint.Version)

workingDirs, err := findWorkingDirs(opts)
Expand All @@ -31,12 +89,12 @@ func (cli *CLI) printVersion(opts Options) int {
fmt.Fprintf(cli.outStream, "working directory: %s\n\n", wd)
}

versions := getPluginVersions(opts)
plugins := getPluginVersions(opts)

for _, version := range versions {
fmt.Fprint(cli.outStream, version)
for _, plugin := range plugins {
fmt.Fprintf(cli.outStream, "+ %s (%s)\n", plugin.Name, plugin.Version)
}
if len(versions) == 0 && opts.Recursive {
if len(plugins) == 0 && opts.Recursive {
fmt.Fprint(cli.outStream, "No plugins\n")
}
return nil
Expand All @@ -46,29 +104,62 @@ func (cli *CLI) printVersion(opts Options) int {
}
}

// Wait for update check to complete and print notification if available
if updateChan != nil {
updateInfo := <-updateChan
if updateInfo != nil && updateInfo.Available {
fmt.Fprintf(cli.outStream, "\nYour version of TFLint is out of date! The latest version\nis %s. You can update by downloading from https://github.com/terraform-linters/tflint/releases\n", updateInfo.Latest)
}
}

return ExitCodeOK
}

func getPluginVersions(opts Options) []string {
// Load configuration files to print plugin versions
func (cli *CLI) printVersionJSON(opts Options, updateInfo *versioncheck.UpdateInfo) int {
// Build output
output := VersionOutput{
Version: tflint.Version.String(),
Plugins: getPluginVersions(opts),
UpdateCheckEnabled: versioncheck.Enabled(),
}

if updateInfo != nil {
output.UpdateAvailable = updateInfo.Available
if updateInfo.Available {
output.LatestVersion = updateInfo.Latest
}
}

// Marshal and print JSON
jsonBytes, err := json.MarshalIndent(output, "", " ")
if err != nil {
log.Printf("[ERROR] Failed to marshal JSON: %s", err)
return ExitCodeError
}

fmt.Fprintln(cli.outStream, string(jsonBytes))
return ExitCodeOK
}

func getPluginVersions(opts Options) []PluginVersion {
cfg, err := tflint.LoadConfig(afero.Afero{Fs: afero.NewOsFs()}, opts.Config)
if err != nil {
log.Printf("[ERROR] Failed to load TFLint config: %s", err)
return []string{}
return []PluginVersion{}
}
cfg.Merge(opts.toConfig())

rulesetPlugin, err := plugin.Discovery(cfg)
if err != nil {
log.Printf("[ERROR] Failed to initialize plugins: %s", err)
return []string{}
return []PluginVersion{}
}
defer rulesetPlugin.Clean()

// Sort ruleset names to ensure consistent ordering
rulesetNames := slices.Sorted(maps.Keys(rulesetPlugin.RuleSets))

versions := []string{}
plugins := []PluginVersion{}
for _, name := range rulesetNames {
ruleset := rulesetPlugin.RuleSets[name]
rulesetName, err := ruleset.RuleSetName()
Expand All @@ -82,8 +173,11 @@ func getPluginVersions(opts Options) []string {
continue
}

versions = append(versions, fmt.Sprintf("+ ruleset.%s (%s)\n", rulesetName, version))
plugins = append(plugins, PluginVersion{
Name: fmt.Sprintf("ruleset.%s", rulesetName),
Version: version,
})
}

return versions
return plugins
}
4 changes: 4 additions & 0 deletions docs/user-guide/environment_variables.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@ Below is a list of environment variables available in TFLint.
- Configure the config file path. See [Configuring TFLint](./config.md).
- `TFLINT_PLUGIN_DIR`
- Configure the plugin directory. See [Configuring Plugins](./plugins.md).
- `TFLINT_DISABLE_VERSION_CHECK`
- Disable version update notifications when running `tflint --version`. Set to `1` to disable.
Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

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

The GITHUB_TOKEN environment variable is used for authenticated GitHub API requests (as mentioned in lines 23-27 of versioncheck/github.go) but is not documented in the environment variables documentation. Consider adding documentation for this optional variable explaining:

  • Its purpose (to increase GitHub API rate limits from 60 to 5000 req/hour)
  • When it's needed
  • How to obtain a token
Suggested change
- Disable version update notifications when running `tflint --version`. Set to `1` to disable.
- Disable version update notifications when running `tflint --version`. Set to `1` to disable.
- `GITHUB_TOKEN`
- (Optional) Used for authenticated GitHub API requests to increase the rate limit from 60 to 5000 requests per hour. This is needed if you encounter rate limit errors when running commands that check for updates or interact with GitHub. You can obtain a token by creating a [GitHub personal access token](https://github.com/settings/tokens); no special scopes are required.

Copilot uses AI. Check for mistakes.
- `GITHUB_TOKEN`
- (Optional) Used for authenticated GitHub API requests when checking for updates and downloading plugins. Increases the rate limit from 60 to 5000 requests per hour. Useful if you encounter rate limit errors. You can obtain a token by creating a [GitHub personal access token](https://github.com/settings/tokens); no special scopes are required.
Comment on lines +13 to +14
Copy link

Copilot AI Nov 27, 2025

Choose a reason for hiding this comment

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

[nitpick] The GITHUB_TOKEN documentation mentions it's used "when checking for updates and downloading plugins" but this placement under version-related variables might be confusing. Consider moving this documentation to a more general location since it already exists elsewhere in the docs and is used for multiple purposes, not just version checking. Also note that the existing plugin code already documents GITHUB_TOKEN usage in the install.go comments (lines 336-344).

Copilot uses AI. Check for mistakes.
- `TFLINT_EXPERIMENTAL`
- Enable experimental features. Note that experimental features are subject to change without notice. Currently only [Keyless Verification](./plugins.md#keyless-verification-experimental) are supported.
- `TF_VAR_name`
Expand Down
86 changes: 86 additions & 0 deletions versioncheck/cache.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,86 @@
package versioncheck

import (
"encoding/json"
"log"
"os"
"path/filepath"
"time"
)

const (
// CacheTTL is how long cached version info is considered valid
CacheTTL = 48 * time.Hour
)

// CacheEntry represents a cached version check result
type CacheEntry struct {
LatestVersion string `json:"latest_version"`
CheckedAt time.Time `json:"checked_at"`
}

// IsExpired returns whether the cache entry has exceeded its TTL
func (c *CacheEntry) IsExpired() bool {
return time.Since(c.CheckedAt) > CacheTTL
}

// loadCache reads and parses the cache file
// Returns nil if cache doesn't exist or is invalid
func loadCache() (*CacheEntry, error) {
cachePath, err := getCachePath()
if err != nil {
return nil, err
}

data, err := os.ReadFile(cachePath)
if err != nil {
if os.IsNotExist(err) {
log.Printf("[DEBUG] No cache file found at %s", cachePath)
return nil, nil
}
return nil, err
}

var entry CacheEntry
if err := json.Unmarshal(data, &entry); err != nil {
log.Printf("[DEBUG] Failed to parse cache file: %s", err)
return nil, err
}

return &entry, nil
}

// saveCache writes the cache entry to disk
func saveCache(entry *CacheEntry) error {
cachePath, err := getCachePath()
if err != nil {
return err
}

// Ensure directory exists
cacheDir := filepath.Dir(cachePath)
if err := os.MkdirAll(cacheDir, 0755); err != nil {
return err
}

data, err := json.MarshalIndent(entry, "", " ")
if err != nil {
return err
}

if err := os.WriteFile(cachePath, data, 0644); err != nil {
return err
}

Comment on lines +71 to +74
Copy link

Copilot AI Nov 27, 2025

Choose a reason for hiding this comment

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

[nitpick] The cache file operations in loadCache and saveCache are not protected against concurrent access. If multiple tflint processes run --version simultaneously, they could race on reading/writing the cache file. Consider using file locking (e.g., syscall.Flock on Unix or similar) or atomic file operations (write to temp file, then rename) to prevent potential corruption. However, since cache corruption would only result in an extra API call (which is not critical), this may be acceptable as-is.

Suggested change
if err := os.WriteFile(cachePath, data, 0644); err != nil {
return err
}
// Write to a temp file, then atomically rename
tmpFile, err := os.CreateTemp(cacheDir, "version_check_cache_*.tmp")
if err != nil {
return err
}
defer func() {
tmpFile.Close()
os.Remove(tmpFile.Name()) // Clean up temp file if rename fails
}()
if _, err := tmpFile.Write(data); err != nil {
return err
}
if err := tmpFile.Sync(); err != nil {
return err
}
if err := tmpFile.Close(); err != nil {
return err
}
if err := os.Rename(tmpFile.Name(), cachePath); err != nil {
return err
}

Copilot uses AI. Check for mistakes.
log.Printf("[DEBUG] Saved version check cache to %s", cachePath)
return nil
}

// getCachePath returns the full path to the cache file using platform-specific cache directory
func getCachePath() (string, error) {
cacheDir, err := os.UserCacheDir()
if err != nil {
return "", err
}
return filepath.Join(cacheDir, "tflint", "version_check_cache.json"), nil
}
54 changes: 54 additions & 0 deletions versioncheck/cache_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
package versioncheck

import (
"testing"
"time"
)

func TestCacheEntry_IsExpired(t *testing.T) {
tests := []struct {
name string
checkedAt time.Time
want bool
}{
{
name: "fresh cache (1 hour old)",
checkedAt: time.Now().Add(-1 * time.Hour),
want: false,
},
{
name: "fresh cache (24 hours old)",
checkedAt: time.Now().Add(-24 * time.Hour),
want: false,
},
{
name: "expired cache (49 hours old)",
checkedAt: time.Now().Add(-49 * time.Hour),
want: true,
},
{
name: "just expired (48 hours + 1 minute)",
checkedAt: time.Now().Add(-48*time.Hour - 1*time.Minute),
want: true,
},
{
name: "just fresh (47 hours)",
checkedAt: time.Now().Add(-47 * time.Hour),
want: false,
},
}

for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
entry := &CacheEntry{
LatestVersion: "0.60.0",
CheckedAt: tt.checkedAt,
}

got := entry.IsExpired()
if got != tt.want {
t.Errorf("CacheEntry.IsExpired() = %v, want %v", got, tt.want)
}
})
}
}
Loading
Loading