Remove secret hints from CLI (#774)
## Summary - Removes the `Hint` column from `arcade secret list` table output — secret hints were removed from the API in [monorepo#322](https://github.com/ArcadeAI/monorepo/pull/322), causing a `KeyError: 'hint'` crash - Adds a test for `print_secret_table` with populated secrets (verifying no hint column) - Bumps version from 1.10.0 → 1.11.0 Closes TOO-444 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
a918eef037
commit
7a28e7f988
3 changed files with 27 additions and 3 deletions
|
|
@ -166,7 +166,6 @@ def print_secret_table(secrets: list[dict]) -> None:
|
||||||
table.add_column("Key", style="cyan")
|
table.add_column("Key", style="cyan")
|
||||||
table.add_column("Type", style="green")
|
table.add_column("Type", style="green")
|
||||||
table.add_column("Description", style="green")
|
table.add_column("Description", style="green")
|
||||||
table.add_column("Hint", style="green")
|
|
||||||
table.add_column("Last Accessed", style="green")
|
table.add_column("Last Accessed", style="green")
|
||||||
table.add_column("Created At", style="green")
|
table.add_column("Created At", style="green")
|
||||||
for secret in secrets:
|
for secret in secrets:
|
||||||
|
|
@ -174,7 +173,6 @@ def print_secret_table(secrets: list[dict]) -> None:
|
||||||
secret["key"],
|
secret["key"],
|
||||||
secret["binding"]["type"],
|
secret["binding"]["type"],
|
||||||
secret["description"],
|
secret["description"],
|
||||||
"..." + secret["hint"] if secret["hint"] else "-",
|
|
||||||
secret["last_accessed_at"] if secret["last_accessed_at"] else "Never",
|
secret["last_accessed_at"] if secret["last_accessed_at"] else "Never",
|
||||||
secret["created_at"],
|
secret["created_at"],
|
||||||
)
|
)
|
||||||
|
|
|
||||||
|
|
@ -24,6 +24,32 @@ class TestPrintSecretTable:
|
||||||
captured = capsys.readouterr()
|
captured = capsys.readouterr()
|
||||||
assert "Tool Secrets" in captured.out
|
assert "Tool Secrets" in captured.out
|
||||||
|
|
||||||
|
def test_print_secret_table_with_secrets(self, capsys):
|
||||||
|
"""Test printing a table with secrets (no hint field)."""
|
||||||
|
secrets = [
|
||||||
|
{
|
||||||
|
"key": "MY_API_KEY",
|
||||||
|
"binding": {"type": "env"},
|
||||||
|
"description": "API key for testing",
|
||||||
|
"last_accessed_at": "2026-01-15T10:00:00Z",
|
||||||
|
"created_at": "2026-01-01T00:00:00Z",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"key": "DB_PASSWORD",
|
||||||
|
"binding": {"type": "env"},
|
||||||
|
"description": "Database password",
|
||||||
|
"last_accessed_at": None,
|
||||||
|
"created_at": "2026-01-02T00:00:00Z",
|
||||||
|
},
|
||||||
|
]
|
||||||
|
print_secret_table(secrets)
|
||||||
|
|
||||||
|
captured = capsys.readouterr()
|
||||||
|
assert "MY_API_KEY" in captured.out
|
||||||
|
assert "DB_PASSWORD" in captured.out
|
||||||
|
assert "Never" in captured.out
|
||||||
|
assert "Hint" not in captured.out
|
||||||
|
|
||||||
|
|
||||||
class TestLoadEnvFile:
|
class TestLoadEnvFile:
|
||||||
"""Tests for load_env_file function."""
|
"""Tests for load_env_file function."""
|
||||||
|
|
|
||||||
|
|
@ -1,6 +1,6 @@
|
||||||
[project]
|
[project]
|
||||||
name = "arcade-mcp"
|
name = "arcade-mcp"
|
||||||
version = "1.10.0"
|
version = "1.11.0"
|
||||||
description = "Arcade.dev - Tool Calling platform for Agents"
|
description = "Arcade.dev - Tool Calling platform for Agents"
|
||||||
readme = "README.md"
|
readme = "README.md"
|
||||||
license = { file = "LICENSE" }
|
license = { file = "LICENSE" }
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue