feat(seo): add meta tags, robots.txt, sitemap.xml; remove .bak files - #1
Merged
Merged
Conversation
Co-authored-by: abrusco <8458577+abrusco@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
abrusco
August 4, 2026 10:28
View session
Owner
|
@copilot resolve the merge conflicts in this pull request |
Owner
|
@copilot resolve the merge conflicts in this pull request |
…SEO improvements Co-authored-by: abrusco <8458577+abrusco@users.noreply.github.com>
Contributor
Author
Contributor
Author
There was a problem hiding this comment.
🟡 Not ready to approve
highlightSql() output is used via innerHTML but does not safely escape non-token content, enabling HTML injection/XSS when rendering log-derived text.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
This PR improves the GitHub Pages (docs/) version of M4 Log Visualizer by adding SEO-focused metadata/assets and refining the standalone UI/behavior, while also removing legacy Node/public artifacts from the repository.
Changes:
- Added SEO meta tags (incl. canonical, Open Graph/Twitter, JSON-LD) to
docs/index.htmland bumped the app version. - Added
docs/robots.txtanddocs/sitemap.xmlfor crawler guidance. - Updated the standalone app (
docs/app.js,docs/styles.css) and removed legacy runtime/assets (server.js,public/*,package.json).
File summaries
| File | Description |
|---|---|
| server.js | Removed legacy Node HTTP server implementation. |
| public/styles.css | Removed legacy non-docs/ stylesheet. |
| public/index.html | Removed legacy non-docs/ HTML entrypoint. |
| public/app.js | Removed legacy non-docs/ app logic. |
| package.json | Removed legacy Node package entrypoint/scripts. |
| docs/version.json | Version bump to 0.9.5. |
| docs/styles.css | Updated styling/layout to new “topbar + split” UI and added new component styles. |
| docs/sitemap.xml | Added sitemap entry for the canonical GitHub Pages URL. |
| docs/robots.txt | Added robots policy and sitemap reference. |
| docs/index.html | Added SEO metadata + structured data; updated UI markup to match new layout. |
| docs/app.js | Updated ignore-object UX, parsing tweaks, search debounce, and UI bindings. |
Review details
Suppressed comments (1)
docs/app.js:186
- Tooltip string is missing Portuguese accents ("ja"/"esta" and "a lista").
elements.ignoreObjectLink.textContent = isIgnored ? 'ignorado' : 'ignorar';
elements.ignoreObjectLink.title = isIgnored ? 'Este objeto ja esta na lista de ignorados' : 'Adicionar objeto atual a lista de ignorados';
- Files reviewed: 6/6 changed files
- Comments generated: 4
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
Comment on lines
114
to
+116
| function addIgnoredObject(name) { | ||
| const options = arguments[1] || {}; | ||
| const deferSelectionMs = Number(options.deferSelectionMs) || 0; |
Comment on lines
+169
to
+173
| elements.ignoreObjectLink.textContent = 'ja ignorado'; | ||
| elements.ignoreObjectLink.title = 'Este objeto ja esta na lista de ignorados'; | ||
| } else { | ||
| elements.ignoreObjectLink.textContent = 'ignorado'; | ||
| elements.ignoreObjectLink.title = 'Objeto adicionado a lista de ignorados'; |
Comment on lines
+8
to
+14
| <!-- SEO --> | ||
| <title>M4 Log Visualizer – Visualizador de Logs Meta4, PeopleNet e Cegid</title> | ||
| <meta name="description" content="Visualizador web standalone de logs Meta4, PeopleNet e Cegid. Analise queries Oracle, objetos Meta4 e resultados de execução diretamente no navegador, sem instalação."> | ||
| <meta name="keywords" content="meta4, peoplenet, cegid, visualizador de logs, análise de logs meta4, sql oracle, m4 log visualizer, ldbinsp0, log analyzer"> | ||
| <meta name="author" content="abrusco"> | ||
| <meta name="theme-color" content="#2c3e50"> | ||
| <meta property="og:title" content="M4 Log Visualizer"> | ||
| <meta property="og:description" content="Visualizador de logs Meta4, PeopleNet e Cegid para análise rápida de SQLs Oracle e objetos Meta4."> | ||
| <meta property="og:url" content="https://abrusco.github.io/M4LogVisualizer/"> | ||
| <meta name="theme-color" content="#1a1a2e"> | ||
| <link rel="canonical" href="https://abrusco.github.io/M4LogVisualizer/"> |
Comment on lines
828
to
829
| return String(sql || '').replace(tokenPattern, (token) => { | ||
| const escaped = escapeHtml(token); |
Co-authored-by: abrusco <8458577+abrusco@users.noreply.github.com>
abrusco
marked this pull request as ready for review
August 4, 2026 10:45
Copilot stopped work on behalf of
abrusco due to an error
August 4, 2026 10:47
Copilot stopped work on behalf of
abrusco due to an error
August 4, 2026 10:48
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
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.
Resumo
Correções de SEO para melhorar a visibilidade da página https://abrusco.github.io/M4LogVisualizer/ no Google.
Mudanças
docs/index.html<title>atualizado com keywords: "M4 Log Visualizer – Visualizador de Logs Meta4, PeopleNet e Cegid"<meta name="description">descritiva adicionada<meta name="keywords">com termos relevantes (meta4, peoplenet, cegid, sql oracle, ldbinsp0…)<meta name="author">e<meta name="theme-color"><link rel="canonical">apontando para a URL canônica do siteog:type,og:url,og:title,og:description)twitter:card,twitter:title,twitter:description)WebApplicationpara structured data / rich resultsdocs/robots.txt(novo)docs/sitemap.xml(novo)Removidos
docs/index.html.bakdocs/styles.css.bak