Skip to content

refactor: improve architecture and documentation#1

Merged
fidalgo merged 4 commits into
mainfrom
improve-architecture
Apr 24, 2026
Merged

refactor: improve architecture and documentation#1
fidalgo merged 4 commits into
mainfrom
improve-architecture

Conversation

@fidalgo
Copy link
Copy Markdown
Contributor

@fidalgo fidalgo commented Apr 24, 2026

How to test

Release notes

Does this change need an explicit changelog or release-note callout beyond the PR title?

Checklist

  • PR title is a Conventional Commit and suitable for squash merge
  • Tests added/updated (if applicable)
  • Docs updated (if applicable)
  • I kept changes focused and easy to review
About the maintainers

Built by Ethos Link, the team behind Reviato.
Capture. Interpret. Act.
Turn guest feedback into clear next steps for your team. Collect private appraisals, spot patterns across reviews, and act before small issues turn into public ones.

fidalgo added 4 commits April 24, 2026 00:25
Validate configuration values close to input and narrow fetcher rescue behavior so programmer errors are not hidden. Add crawler-level failure pages for per-URL fetch exceptions.

Move shared Rake task execution into Crawlscope::RakeTasks, clarify task descriptions, and document REPORT_PATH as trusted operator input.

Refactor the links rule into named extraction and target-resolution steps, and make sitemap structured data validation report missing JSON-LD or microdata with an actionable message.

Tests: bundle exec rake test; RUBOCOP_CACHE_ROOT=/tmp/rubocop-cache bundle exec standardrb
@fidalgo fidalgo changed the title Improve architecture refactor: improve architecture and documentation Apr 24, 2026
@fidalgo fidalgo merged commit 4042820 into main Apr 24, 2026
4 checks passed
@fidalgo fidalgo deleted the improve-architecture branch April 24, 2026 09:43
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