Skip to content

chore: enable marketplace plugins and record docs.spring.io redirect finding - #251

Draft
amondnet wants to merge 2 commits into
mainfrom
chore/settings-plugins
Draft

amondnet wants to merge 2 commits into
mainfrom
chore/settings-plugins

Conversation

@amondnet

@amondnet amondnet commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Configuration and knowledge housekeeping only — no source, test, or build changes.

  • Enable seven newly installed marketplace plugins in .claude/settings.json: github-actions@passionfactory, tidy-first@passionfactory, context@passionfactory, research@passionfactory, claude-code-docs@pleaseai, kb@passionfactory, memory-management@passionfactory.
  • Record a project-scope agent memory entry documenting that docs.spring.io 301-collapses a patch path segment to its minor version (and older minors stop serving assets entirely), which is why imageBaseFor in scripts/lib/upstream-sources.ts builds reference-site URLs from raw.githubusercontent.com/.../v<version>/... instead of a docs.spring.io path. Also notes the probe that catches the collapse: curl without -L, since following the redirect hides it behind a 200.

Test plan

  • N/A — configuration and agent-memory content only, no runtime behavior affected.

Summary by cubic

Enables seven newly installed marketplace plugins in .claude/settings.json and records an agent memory entry documenting why docs.spring.io cannot be used for version-pinned asset URLs.

Written for commit 762032f. Summary will update on new commits.

…mory

Project-scope agent memory is shared with the team, so it belongs in git.

The entry documents why `imageBaseFor` points at release-tag raw URLs rather
than the reference site: `docs.spring.io` 301-collapses a patch segment to its
minor, and older minors stop serving assets entirely, so a URL built from a
catalog version looks pinned but is not. It also records the check that catches
it — `curl` without `-L`, since a redirect-following probe hides the collapse
behind a 200.
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@sonarqubecloud

Copy link
Copy Markdown

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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.

1 participant