docs: document sourcecraft inspector
Metadata
Current evaluation
Merged PR #219 (commit f4a5dac) by dariuszd21 added docs/reference/sourcecraft.rst documenting the Sourcecraft inspector added in #217, closing CRAFT-3337. File later revised by #604.
Resolution: closed resolved
Reason: The PR was merged into main as commit f4a5dac, adding docs/reference/sourcecraft.rst (listed in docs/reference/index.rst) describing craft.sourcecraft behaviour, rejection reasons and artifact metadata, fulfilling the CRAFT-3337 documentation task after approval by two reviewers.
Evaluated by: mr-cal-bot
Issue body
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #219 (commit f4a5dac) by dariuszd21 added docs/reference/sourcecraft.rst documenting the Sourcecraft inspector added in #217, closing CRAFT-3337. File later revised by #604. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #219, documenting the Sourcecraft inspector and completing CRAFT-3337 after approval by two reviewers. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged documentation for the sourcecraft inspector. Approved by two reviewers, the changes were successfully integrated, resolving CRAFT-3337. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/fetch-service#217
(confidence 90%)
PR #217 / commit 4e14f93 'feat: add sourcecraft inspector' introduced the craft.sourcecraft inspector that this PR documents.
-
Related To:
canonical/fetch-service#229
(confidence 45%)
Follow-up fix removing text/html inspection in the sourcecraft inspector, recorded in docs/changelog.rst.
-
Related To:
canonical/fetch-service#604
(confidence 40%)
Commit 3ed443f 'docs: update inspectors reference' later touched docs/reference/sourcecraft.rst, superseding parts of this doc.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #217 feat: add sourcecraft inspector | fetch-service | merged | Merged PR #217 as commit 4e14f93, adding Sourcecraft inspection and repository-cloning support; it declared #211 fixed and followed prerequisite PR #216. | |
| #222 test: enable sourcecraft inspector tests | fetch-service | merged | Merged PR #222 as commit 6e68dc0, re-enabling Sourcecraft inspector tests accidentally disabled during refactoring; DEB tests were deferred to separate work. | |
| #228 Sourcecraft inspector is too restrictive | fetch-service | closed | Fixed by merged PR #229 (commit aaa40a4): Sourcecraft no longer inspects HTML responses, preventing unrelated permissive-mode artifacts from being rejected. | |
| #229 fix: remove inspecting text/html in sourcecraft inspector | fetch-service | merged | Merged PR #229 fixed accidental Sourcecraft inspection of HTML by removing text/html inspection; released in v0.2.1 via commit aaa40a4. | |
| #117 docs: add git inspectors reference documentation | fetch-service | merged | Git inspectors reference documentation for CRAFT-2639 was merged in #117 via commit 61e0983 after approval by two reviewers. |