Conversation
…omPulse link
Two corrections found running this skill against the live connector.
The monthly sales trend is an estimate, not real Mercado Livre data. The skill
lists "the sales trend" among the real MeLi figures alongside the 60- and
365-day counts. That was accurate for salesTrend, the API run-rate derived from
completed-sale counts — but pulse://rules AP-8 forbids using that field as a
seller's trend, since it is NULL for roughly 82% of sellers, and directs every
client to MlbSellersSnapshot.monthlySalesGrowth, which is the month-over-month
change in JoomPulse's estimated sales. The field changed; the disclaimer did
not. A live run told a seller its -7.0% momentum was verified Mercado Livre
data.
Sellers have no JoomPulse page. The snapshot row list asks for "Link JoomPulse
do vendedor". pulse://overview rule 10 restricts the beginner-products/{id}
template to listing ids and skips seller grain, so any URL built from a shopId
404s. The Shopee bullet below now reads "no JoomPulse dashboard link for Shopee
either", so it no longer implies Mercado Livre has one.
No behavioural change beyond those two lines.
fiharet34
force-pushed
the
fix/seller-trend-is-an-estimate
branch
from
September 11, 2026 11:05
61bbcea to
835d7da
Compare
fiharet34
marked this pull request as ready for review
September 14, 2026 12:58
fiharet34
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two corrections found running this skill against the live connector.
The monthly sales trend is an estimate, not real Mercado Livre data. The skill lists "the sales trend" among the real MeLi figures alongside the 60- and 365-day counts. That was accurate for
salesTrend, the API run-rate derived from completed-sale counts — butpulse://rulesAP-8 forbids using that field as a seller's trend (NULL for roughly 82% of sellers) and directs every client toMlbSellersSnapshot.monthlySalesGrowth, which is the month-over-month change in JoomPulse's estimated sales. The field changed; the disclaimer did not. A live run told a seller its −7,0% momentum was verified Mercado Livre data.Sellers have no JoomPulse page. The snapshot row list asks for "Link JoomPulse do vendedor".
pulse://overviewrule 10 restricts thebeginner-products/{id}template to listing ids and skips seller grain, so any URL built from ashopId404s. The Shopee bullet below now reads "no JoomPulse dashboard link for Shopee either", so it no longer implies Mercado Livre has one.No behavioural change beyond those two lines.