Skip to content

Add koreanpulse β€” Korean equity intelligence MCP#42

Open
whdrnr2583-cmd wants to merge 2 commits into
jmanhype:mainfrom
whdrnr2583-cmd:add-koreanpulse
Open

Add koreanpulse β€” Korean equity intelligence MCP#42
whdrnr2583-cmd wants to merge 2 commits into
jmanhype:mainfrom
whdrnr2583-cmd:add-koreanpulse

Conversation

@whdrnr2583-cmd

Copy link
Copy Markdown

Description

Adds koreanpulse to MCP Servers section.

  • Korean equity intelligence MCP β€” DART filings, foreign-holder 5%-rule
    flows (BlackRock / Vanguard / Norges / GIC + 16 more), Korean activist
    filings (KCGI / Align / ValueAct / Elliott), KRX industry news.
  • Free public daily snapshot at koreanpulse.dev/today.
  • 7 MCP tools, AGPL-3.0 + commercial Cloud waitlist.
  • Hosted gateway via Smithery (no local install required).

Repo: https://github.com/whdrnr2583-cmd/koreanpulse

Type of contribution

  • Plugin/Extension
  • MCP Server
  • Editor Plugin/Integration
  • Documentation/Resource
  • Other (please specify)

Submission checklist

  • I have read the contribution guidelines (if any)
  • The item is awesome and relevant to Claude Code
  • The link is working and leads to the correct resource
  • The description is clear and concise
  • The item is added in the appropriate category
  • The item follows the existing format
  • I have verified that this item is not already in the list

Additional context

@qodo-code-review

Copy link
Copy Markdown

Review Summary by Qodo

Add koreanpulse Korean equity intelligence MCP server

✨ Enhancement

Grey Divider

Walkthroughs

Description
β€’ Adds koreanpulse MCP server to the list
β€’ Provides Korean equity intelligence with DART filings
β€’ Includes foreign-holder flows and activist filings tracking
β€’ Offers free daily snapshot and 7 MCP tools
Diagram
flowchart LR
  A["MCP Servers List"] -- "adds new entry" --> B["koreanpulse"]
  B -- "provides" --> C["Korean Equity Intelligence"]
  C -- "includes" --> D["DART Filings & Flows"]
  C -- "includes" --> E["Activist Filings & News"]
Loading

Grey Divider

File Changes

1. README.md ✨ Enhancement +1/-0

Add koreanpulse MCP server table entry

β€’ Adds koreanpulse entry to MCP Servers table
β€’ Includes repository link and authentication requirements
β€’ Describes features: DART filings, foreign-holder flows, activist filings, KRX news
β€’ References free daily snapshot and licensing model

README.md


Grey Divider

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented May 5, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (1) πŸ“˜ Rule violations (1) πŸ“Ž Requirement gaps (0)

Grey Divider


Action required

1. Stale MCP server count βœ“ Resolved 🐞 Bug ≑ Correctness
Description
README.md's Status table still reports 5 MCP servers, but the MCP Servers table now lists 6 entries
after adding koreanpulse, making the published metrics incorrect.
Code

README.md[43]

+| [koreanpulse](https://github.com/whdrnr2583-cmd/koreanpulse) | License key + DART API key | Korean equity intelligence β€” DART filings, foreign-holder 5%-rule flows (BlackRock / Vanguard / Norges / GIC + 16 more), Korean activist filings (KCGI / Align / ValueAct / Elliott), KRX industry news. Free public daily snapshot at koreanpulse.dev/today. |
Evidence
The Status table metric is not updated to reflect the newly added MCP server row, creating an
internal inconsistency in README.md.

README.md[7-13]
README.md[34-43]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The README `## Status` metrics are now inconsistent with the `## MCP Servers` list: the list gained a new row, but the count was not incremented.
### Issue Context
Readers will see conflicting information: `MCP servers | 5 |` while the table contains 6 MCP servers.
### Fix Focus Areas
- README.md[7-13]
- README.md[34-43]
### Suggested change
Update the Status row to `| MCP servers | 6 |` (and consider updating `Last updated` if that field is meant to reflect changes like this).

β“˜ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Remediation recommended

2. koreanpulse row too verbose πŸ“˜ Rule violation βš™ Maintainability
Description
The new koreanpulse entry uses a multi-sentence, unusually long Coverage cell compared to the
existing MCP Servers table rows, which are short phrases. This diverges from the established README
table pattern and reduces scanability/consistency.
Code

README.md[43]

+| [koreanpulse](https://github.com/whdrnr2583-cmd/koreanpulse) | License key + DART API key | Korean equity intelligence β€” DART filings, foreign-holder 5%-rule flows (BlackRock / Vanguard / Norges / GIC + 16 more), Korean activist filings (KCGI / Align / ValueAct / Elliott), KRX industry news. Free public daily snapshot at koreanpulse.dev/today. |
Evidence
PR Compliance ID 5 requires new/modified content to match existing repository patterns. In
README.md, other MCP server Coverage cells are concise phrases, while the added koreanpulse
entry includes a long description with multiple clauses and a full additional sentence (`Free public
daily snapshot ...`).

CLAUDE.md
README.md[43-43]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The `koreanpulse` MCP Servers table row has a `Coverage` description that is significantly longer and more sentence-like than the other entries, diverging from the README table’s established concise style.
## Issue Context
Other `Coverage` cells in the MCP Servers table are short phrases (e.g., `Jira, Confluence`, `Repos, issues, PRs, workflows`). The new entry includes a long, multi-clause description plus an extra sentence.
## Fix Focus Areas
- README.md[43-43]

β“˜ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


3. Non-clickable snapshot URL 🐞 Bug βš™ Maintainability
Description
The added koreanpulse row includes koreanpulse.dev/today as plain text without a scheme or
markdown link, so it may not render as a clickable URL and is inconsistent with other README links.
Code

README.md[43]

+| [koreanpulse](https://github.com/whdrnr2583-cmd/koreanpulse) | License key + DART API key | Korean equity intelligence β€” DART filings, foreign-holder 5%-rule flows (BlackRock / Vanguard / Norges / GIC + 16 more), Korean activist filings (KCGI / Align / ValueAct / Elliott), KRX industry news. Free public daily snapshot at koreanpulse.dev/today. |
Evidence
The koreanpulse row includes a bare domain path instead of the markdown link style used elsewhere in
the README.

README.md[43-43]
README.md[56-65]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The README includes `koreanpulse.dev/today` as plain text (no `https://` and not wrapped in markdown link syntax), which often won’t be clickable.
### Issue Context
Other resources in the README are formatted as markdown links.
### Fix Focus Areas
- README.md[43-43]
### Suggested change
Replace the plain text with a proper link, e.g. `Free public daily snapshot at [koreanpulse.dev/today](https://koreanpulse.dev/today).`

β“˜ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

Comment thread README.md
Qodo review caught the Status metrics row stale after adding koreanpulse.
@whdrnr2583-cmd

whdrnr2583-cmd commented May 5, 2026 via email

Copy link
Copy Markdown
Author

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