Add journal-intelligence skill: live guidelines, AI policy, LaTeX template fetch#109
Add journal-intelligence skill: live guidelines, AI policy, LaTeX template fetch#109eniktab wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e919f83de1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -0,0 +1,158 @@ | |||
| --- | |||
| name: journal-intelligence | |||
There was a problem hiding this comment.
Add this skill to the marketplace skills list
This commit creates journal-intelligence, but the plugin manifest still uses an explicit allowlist of skill directories in .claude-plugin/marketplace.json and does not include ./scientific-skills/journal-intelligence. In that packaging flow, the new skill is not discoverable/installable even though this file exists, so users cannot actually invoke the feature being introduced.
Useful? React with 👍 / 👎.
|
|
||
| ## Integration with scientific-writing | ||
|
|
||
| `journal-intelligence` is designed to run as Phase 0 before the `scientific-writing` skill. The `journal_profile.yaml` it produces is consumed directly by `scientific-writing` to set all word limits, citation format, required sections, and template class — overriding any defaults in that skill. |
There was a problem hiding this comment.
Remove or implement the claimed auto-integration
This line states that scientific-writing directly consumes journal_profile.yaml, but there is no corresponding reference to journal-intelligence or journal_profile.yaml in scientific-skills/scientific-writing/SKILL.md (repo-wide search only finds these terms in this new file). As written, users can reasonably assume automatic Phase-0 execution and skip an explicit run, which means the live guideline step may be missed.
Useful? React with 👍 / 👎.
…w with scientific-writing
|
Thanks for the review — both points addressed in the latest commit (e77b2fd): P1 — marketplace.json: Added P2 — Auto-integration claim: Removed the misleading statement. The section now explicitly documents a two-step manual workflow — run |
Gonzih
left a comment
There was a problem hiding this comment.
Overall this is a well-structured and genuinely useful skill — the motivation (hardcoded journal requirements go stale) is sound, and the journal_profile.yaml schema captures the right fields. A few things to address before approval:
Needs clarification / fixes
-
allowed-toolsin frontmatter is non-standard for this repo. The existing skills do not use anallowed-toolskey in YAML frontmatter — this appears to be a field specific to your standalone repo. Check whether the K-Dense marketplace format supports or ignores this field. If it's ignored, it's harmless clutter; if it causes a parse error, it needs to be removed. -
WebSearch/WebFetch dependency is a hard runtime requirement. This skill only works if the agent runtime has WebSearch and WebFetch tools available. The fallback section acknowledges network failures but not the case where these tools simply aren't configured. A brief note under "When to Use" that this skill requires
WebSearchandWebFetchto be enabled in the agent's tool set would set accurate expectations for users. -
URL patterns are partially outdated or publisher-specific. The listed URL patterns for Elsevier (
/journals/[journal]/[issn]/guide-for-authors) and PLOS (/journals/[journal]/s/submission-guidelines) are reasonable starting points, but Elsevier in particular has a highly variable URL structure. Framing these as "example patterns to try" rather than authoritative patterns would be more accurate. -
The self-promotion link at the bottom (
Part of the claude-scientific-paper-writer suite) — maintainers may want to remove or reframe this, as other contributed skills don't link to external contributor projects from within the skill body.
What's good
- The
journal_profile.yamlschema is comprehensive and well thought out — the 20+ fields cover the full spectrum of what a writer actually needs. - Separate
ai_policy.mdoutput for AI disclosure language is a smart design choice given how rapidly those policies are changing. - The two-step workflow description (journal-intelligence → scientific-writing) with explicit handoff via the YAML file is clear and practical.
- The fallback behavior with
[VERIFY]flags is a responsible approach. - Correct registration in
marketplace.json, placed logically beforescientific-writing.
Addressing points 1–2 would make this ready to approve.
- Remove non-standard allowed-tools from frontmatter - Document WebSearch/WebFetch as hard runtime requirements under When to Use - Reframe URL patterns as examples with explicit fallback to WebSearch - Remove self-promotion footer link
Gonzih
left a comment
There was a problem hiding this comment.
Solid, well-motivated skill. The core insight — that journal requirements change frequently enough that hardcoding them is brittle — is well-supported by the rationale table (Nature Biotech 2023, AI policy changes 2023–2025, PLOS 2024). Making guideline fetch a discrete Phase 0 before scientific-writing is the right architectural decision.
Specific things that work well:
- The known URL pattern table for major publishers (Nature, Elsevier, Cell Press, PLOS, etc.) gives Claude a fast-path before falling back to WebSearch.
- Storing results as
journal_profile.yamlwith 20+ structured fields (word limits, methods placement, citation style, AI disclosure) makes the output machine-readable for downstream steps. - The AI policy extraction step (
ai_policy.md) is timely given how rapidly publisher AI policies are evolving. - Fallback behavior with
[VERIFY]flags when network is unavailable is the right safe default. - Tool dependencies (WebSearch + WebFetch) are clearly called out upfront.
Minor suggestions (non-blocking):
- Consider adding a brief
## Example Outputsnippet showing what a populatedjournal_profile.yamllooks like — this would help users understand the schema before running the skill. - The
marketplace.jsonplacement betweenscientific-critical-thinkingandscientific-writingis logical; just verify the maintainer is OK with the insertion point.
LGTM overall.
Summary
Adds a
journal-intelligenceskill that fetches live author guidelines, AI authorship policy, and LaTeX template for any journal before writing — filling a gap in the currentscientific-writingskill, which uses hardcoded format requirements that go stale.What it adds
journal_profile.yaml(20+ fields: word limits, abstract format, figure limits, methods placement, citation style, required special sections)[VERIFY]flags when network is unavailableWhy hardcoded limits are a problem
Submitting with stale requirements causes desk rejection. The fix is always fetch live.
Integration with existing scientific-writing skill
journal-intelligenceruns as Phase 0 beforescientific-writing. Thejournal_profile.yamlit produces is consumed directly to set word limits, citation format, required sections, and template class — overriding defaults.Full standalone repo
https://github.com/eniktab/claude-scientific-paper-writer