Skip to content

docs: fix HINDSIGHT_API_LLM_MAX_RETRIES default (10 → 3)#1137

Merged
nicoloboschi merged 2 commits intovectorize-io:mainfrom
r266-tech:docs-fix-max-retries-default
Apr 18, 2026
Merged

docs: fix HINDSIGHT_API_LLM_MAX_RETRIES default (10 → 3)#1137
nicoloboschi merged 2 commits intovectorize-io:mainfrom
r266-tech:docs-fix-max-retries-default

Conversation

@r266-tech
Copy link
Copy Markdown
Contributor

Fixes #1136

PR #1121 reduced DEFAULT_LLM_MAX_RETRIES from 10 to 3 in config.py, but the configuration reference table still showed the old default.

Changes

  • hindsight-docs/docs/developer/configuration.md line 169: 103
  • skills/hindsight-docs/references/developer/configuration.md line 169: synced

Copy link
Copy Markdown
Collaborator

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

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

thanks!

@nicoloboschi nicoloboschi merged commit f75251f into vectorize-io:main Apr 18, 2026
45 of 46 checks passed
r266-tech added a commit to r266-tech/hindsight that referenced this pull request Apr 20, 2026
Mirror change from hindsight-docs/docs/developer/admin-cli.md so the
docs skill reference stays in sync (matches the pattern set by vectorize-io#1137).
nicoloboschi pushed a commit that referenced this pull request Apr 21, 2026
* docs(admin-cli): document decommission-workers and worker-status

PR #1165 added two new admin CLI commands (decommission-workers,
worker-status) but admin-cli.md was not updated. Readers scanning the
Commands section could only find the singular decommission-worker.

Added dedicated sections for each new command following the existing
style (Arguments/Options/Examples/When to Use). Pure docs, mirrors
behavior documented in typer command help strings.

* docs(admin-cli skill): sync decommission-workers and worker-status

Mirror change from hindsight-docs/docs/developer/admin-cli.md so the
docs skill reference stays in sync (matches the pattern set by #1137).
r266-tech added a commit to r266-tech/hindsight that referenced this pull request Apr 21, 2026
Follow-up to vectorize-io#1168: update_bank now accepts config_updates with all
bank-configurable fields (reflect_mission, retain_*, disposition_*,
entity_labels, recall_*, mcp_enabled_tools, etc.). Existing docs only
showed name + mission; callers had to read mcp_tools.py to discover the
full surface.

Mirrored to skills/hindsight-docs/references/developer/mcp-server.md per
the dual-doc convention (vectorize-io#1137).
nicoloboschi pushed a commit that referenced this pull request Apr 21, 2026
#1183)

* docs(mcp): document update_bank config_updates and configurable fields

Follow-up to #1168: update_bank now accepts config_updates with all
bank-configurable fields (reflect_mission, retain_*, disposition_*,
entity_labels, recall_*, mcp_enabled_tools, etc.). Existing docs only
showed name + mission; callers had to read mcp_tools.py to discover the
full surface.

Mirrored to skills/hindsight-docs/references/developer/mcp-server.md per
the dual-doc convention (#1137).

* docs(mcp): mirror update_bank config_updates docs to skills reference
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.

docs: HINDSIGHT_API_LLM_MAX_RETRIES default shown as 10, actual is 3

2 participants