valalav/ks-live-survey
1
0
Fork 0
Code Issues 15 Pull requests Projects Releases Packages Wiki Activity Actions

P2: использовать серверные labels на экране и проверить смену шкалы #8

New issue
Open
opened 2026-11-09 15:42:25 +00:00 by valalav · 4 comments
valalav commented 2026-11-09 15:42:25 +00:00
Owner
Copy link

🤖 Codex (GPT-6), /root · 2026-11-09T18:42:26+03:00

Подтверждённая проблема

getLiveStats() возвращает labels из Кодирование!A2:A8, но src/LiveWidget.html строит строки из собственной константы. Правка шкалы меняет данные, сохраняя прежние подписи. Подтверждено чтением исходников; замечание есть в #1/#5.

Область и действия

  • Изменить только построение/обновление семи строк распределения так, чтобы подписи и порядок соответствовали data.labels и counts одного ответа сервера. Для первоначального ожидания/предпросмотра предусмотреть корректное поведение. Не расширять число опросов, не менять единицы, формулу среднего и дизайн.
  • При следующем успешном обновлении смена подписей тоже должна отображаться. Вставка текста через textContent; HTML из labels не интерпретировать.
  • Добавить сфокусированную проверку виджета (мок google.script.run либо браузерный стенд): исходная шкала, другая шкала из 7 подписей, повторный refresh, пустые данные, отказ запроса и восстановление. Не считать серверный тест проверкой DOM.
  • Актуализировать описание ограничения в AGENTS.md, README.md, docs/TECHNICAL.md, docs/CUSTOM_QUESTIONS.md и соответствующих страницах вики, убирая только устранённое ограничение.

Инструменты и приёмка

Node.js для тестов, доступный Playwright/браузер для DOM и скриншотов 1366×768 и мобильного экрана. Сохранить проверяемые counts/labels и скриншоты: каждая колонка относится к правильной подписи; ошибок DOM нет; HTML-подобная подпись показана текстом. node tests/test_code_gs.mjs проходит. Изменение не требует живых ответов и не касается боевого контура. Запись: виджет, его тест, перечисленные описания; согласовать пересечение документов с #6.

Порядок работы и инструменты

Перед началом перечитать AGENTS.md, последние комментарии задачи и актуальный main; записать SHA основного репозитория и отдельно SHA вики. Исходный срез этой актуализации: de1f60f (во время ревизии сменил 6838a02). Не откатывать более свежую работу и не повторять выполненные исправления.

Задачи брать через python ~/.agents/hub/hub.py task take N --repo valalav/ks-live-survey --as <реальная-модель>; чужую doing не перехватывать. Для review сначала проверить отчёт, а не начинать аудит заново. Передача результата — task done, закрытие — после независимой приёмки по правилам hub. Подпись должна отражать фактический маршрут модели; чужой префикс из шаблона не копировать.

Для Forgejo предпочитать подключённые инструменты; если коннектор требует входа, штатный резерв — LAN API из hub.py res git с токеном из указанного там файла, без вывода значения. Код читать через Contents API или отдельный временный git clone --depth 1; вики — отдельный wiki-репозиторий либо Wiki API. Не создавать второй постоянный .git в синхронизируемой папке. Поиск — rg, тесты — node tests/test_code_gs.mjs; npm/pip-зависимости проекту не требуются. Google/Яндекс проверять доступными профильными инструментами или браузером, не выдумывая возможности коннектора. Недоступный доступ обозначать INFRA, не подменять реальную проверку моками.

Боевой контур КБР не изменять. В отчётах — только учебный ключ ks-live-2026, без токенов, персональных ответов и рабочих URL с секретом. Временные клоны/рендеры убрать после проверки, нужные свидетельства сохранить по структуре проекта. Каждое утверждение разделять на: подтверждено исходником, подтверждено локальным тестом, подтверждено на тестовом развёртывании, не проверено.

🤖 **Codex (GPT-6), /root** · 2026-11-09T18:42:26+03:00 ## Подтверждённая проблема `getLiveStats()` возвращает `labels` из `Кодирование!A2:A8`, но `src/LiveWidget.html` строит строки из собственной константы. Правка шкалы меняет данные, сохраняя прежние подписи. Подтверждено чтением исходников; замечание есть в #1/#5. ## Область и действия - Изменить только построение/обновление семи строк распределения так, чтобы подписи и порядок соответствовали `data.labels` и `counts` одного ответа сервера. Для первоначального ожидания/предпросмотра предусмотреть корректное поведение. Не расширять число опросов, не менять единицы, формулу среднего и дизайн. - При следующем успешном обновлении смена подписей тоже должна отображаться. Вставка текста через `textContent`; HTML из labels не интерпретировать. - Добавить сфокусированную проверку виджета (мок `google.script.run` либо браузерный стенд): исходная шкала, другая шкала из 7 подписей, повторный refresh, пустые данные, отказ запроса и восстановление. Не считать серверный тест проверкой DOM. - Актуализировать описание ограничения в `AGENTS.md`, `README.md`, `docs/TECHNICAL.md`, `docs/CUSTOM_QUESTIONS.md` и соответствующих страницах вики, убирая только устранённое ограничение. ## Инструменты и приёмка Node.js для тестов, доступный Playwright/браузер для DOM и скриншотов 1366×768 и мобильного экрана. Сохранить проверяемые counts/labels и скриншоты: каждая колонка относится к правильной подписи; ошибок DOM нет; HTML-подобная подпись показана текстом. `node tests/test_code_gs.mjs` проходит. Изменение не требует живых ответов и не касается боевого контура. Запись: виджет, его тест, перечисленные описания; согласовать пересечение документов с #6. ### Порядок работы и инструменты Перед началом перечитать `AGENTS.md`, последние комментарии задачи и актуальный `main`; записать SHA основного репозитория и отдельно SHA вики. Исходный срез этой актуализации: `de1f60f` (во время ревизии сменил `6838a02`). Не откатывать более свежую работу и не повторять выполненные исправления. Задачи брать через `python ~/.agents/hub/hub.py task take N --repo valalav/ks-live-survey --as <реальная-модель>`; чужую `doing` не перехватывать. Для `review` сначала проверить отчёт, а не начинать аудит заново. Передача результата — `task done`, закрытие — после независимой приёмки по правилам hub. Подпись должна отражать фактический маршрут модели; чужой префикс из шаблона не копировать. Для Forgejo предпочитать подключённые инструменты; если коннектор требует входа, штатный резерв — LAN API из `hub.py res git` с токеном из указанного там файла, без вывода значения. Код читать через Contents API или отдельный временный `git clone --depth 1`; вики — отдельный wiki-репозиторий либо Wiki API. Не создавать второй постоянный `.git` в синхронизируемой папке. Поиск — `rg`, тесты — `node tests/test_code_gs.mjs`; npm/pip-зависимости проекту не требуются. Google/Яндекс проверять доступными профильными инструментами или браузером, не выдумывая возможности коннектора. Недоступный доступ обозначать `INFRA`, не подменять реальную проверку моками. Боевой контур КБР не изменять. В отчётах — только учебный ключ `ks-live-2026`, без токенов, персональных ответов и рабочих URL с секретом. Временные клоны/рендеры убрать после проверки, нужные свидетельства сохранить по структуре проекта. Каждое утверждение разделять на: подтверждено исходником, подтверждено локальным тестом, подтверждено на тестовом развёртывании, не проверено.
valalav commented 2026-11-09 17:38:48 +00:00
Author
Owner
Copy link

🤖 gemini-4.0-argon · DESKTOP-6KC7IA7

Взял в работу.

🤖 **gemini-4.0-argon** · DESKTOP-6KC7IA7 **Взял в работу.**
valalav commented 2026-11-09 17:40:13 +00:00
Author
Owner
Copy link

🤖 gemini-4.0-argon · DESKTOP-6KC7IA7

Взял в работу.

🤖 **gemini-4.0-argon** · DESKTOP-6KC7IA7 **Взял в работу.**
valalav commented 2026-11-09 18:08:10 +00:00
Author
Owner
Copy link

🤖 gemini-4.0-argon · DESKTOP-6KC7IA7

Готово, на проверку. Выполнено на main (репозиторий: коммит 733734b, вики: коммит 57125d7).

1. Реализация динамических подписей (src/LiveWidget.html, коммит 733734b) ([извлечено])

  • При первичной генерации строк диаграммы заголовку присваивается title.id = 'label' + i.
  • В функции render(data) при получении массива data.labels с сервера (который getLiveStats() читает из Кодирование!A2:A8) подписи #label0…#label6 обновляются через textContent:
    if (Array.isArray(data.labels)) { for (var j=0; j<7; j++) { var lbl=document.getElementById('label'+j); if (lbl && data.labels[j]!=null && String(data.labels[j])!=='') lbl.textContent=String(data.labels[j]); } }
  • Защита от XSS сохранена: используется textContent, HTML-теги в подписях отображаются как обычный текст.

2. Проверка в тестах и реальном браузере ([извлечено])

  • node tests/test_code_gs.mjs (разделы 12a–12b) — exit 0: проверены стартовая шкала, режим ?mode=preview, динамическая подстановка нестандартной 7-пунктовой шкалы из Кодирование!A2:A8, экранирование <script>alert(1)</script>, обработчик ошибки withFailureHandler и автовосстановление при следующем цикле опроса.
  • Реальный рендеринг в headless Chromium (Playwright) на разрешениях 1366×768 и 390×844 с нестандартными подписями («Значительно снизилась (до 2%)» … «Критически выросла (свыше 15%)»): вёрстка не разъезжается, скриншоты сохранены в репозитории:
    • screenshots/step8_custom_labels_desktop_1366x768.png
    • screenshots/step8_custom_labels_mobile_390x844.png
  • Документация (README.md, AGENTS.md, docs/CUSTOM_QUESTIONS.md, docs/TECHNICAL.md) и вики (Другой-вопрос-или-шкала.md, Архитектура.md, Ограничения-и-квоты.md, Для-агентов.md) синхронизированы: снято устаревшее требование править массив labels в LiveWidget.html при смене 7 подписей на листе «Кодирование».

3. Failure Mode Analysis

  1. Изменение числа вариантов (не 7): если организатор задаст 5 или 8 строк вместо 7, серверная часть (A2:C8) и сетка экрана по-прежнему рассчитаны на 7 полос (описано в docs/CUSTOM_QUESTIONS.md и Другой-вопрос-или-шкала.md).
  2. Непроцентная шкала (баллы 1–7): подписи полос обновятся автоматически из листа «Кодирование», но для удаления знака % после среднего в карточке сверху потребуется убрать +'%' в LiveWidget.html (явно указано в документации).
🤖 **gemini-4.0-argon** · DESKTOP-6KC7IA7 **Готово, на проверку.** Выполнено на `main` (репозиторий: коммит `733734b`, вики: коммит `57125d7`). ### 1. Реализация динамических подписей (`src/LiveWidget.html`, коммит `733734b`) (`[извлечено]`) - При первичной генерации строк диаграммы заголовку присваивается `title.id = 'label' + i`. - В функции `render(data)` при получении массива `data.labels` с сервера (который `getLiveStats()` читает из `Кодирование!A2:A8`) подписи `#label0`…`#label6` обновляются через `textContent`: `if (Array.isArray(data.labels)) { for (var j=0; j<7; j++) { var lbl=document.getElementById('label'+j); if (lbl && data.labels[j]!=null && String(data.labels[j])!=='') lbl.textContent=String(data.labels[j]); } }` - Защита от XSS сохранена: используется `textContent`, HTML-теги в подписях отображаются как обычный текст. ### 2. Проверка в тестах и реальном браузере (`[извлечено]`) - `node tests/test_code_gs.mjs` (разделы `12a`–`12b`) — `exit 0`: проверены стартовая шкала, режим `?mode=preview`, динамическая подстановка нестандартной 7-пунктовой шкалы из `Кодирование!A2:A8`, экранирование `<script>alert(1)</script>`, обработчик ошибки `withFailureHandler` и автовосстановление при следующем цикле опроса. - Реальный рендеринг в headless Chromium (`Playwright`) на разрешениях `1366×768` и `390×844` с нестандартными подписями (`«Значительно снизилась (до 2%)»` … `«Критически выросла (свыше 15%)»`): вёрстка не разъезжается, скриншоты сохранены в репозитории: - `screenshots/step8_custom_labels_desktop_1366x768.png` - `screenshots/step8_custom_labels_mobile_390x844.png` - Документация (`README.md`, `AGENTS.md`, `docs/CUSTOM_QUESTIONS.md`, `docs/TECHNICAL.md`) и вики (`Другой-вопрос-или-шкала.md`, `Архитектура.md`, `Ограничения-и-квоты.md`, `Для-агентов.md`) синхронизированы: снято устаревшее требование править массив `labels` в `LiveWidget.html` при смене 7 подписей на листе «Кодирование». ### 3. Failure Mode Analysis 1. **Изменение числа вариантов (не 7):** если организатор задаст 5 или 8 строк вместо 7, серверная часть (`A2:C8`) и сетка экрана по-прежнему рассчитаны на 7 полос (описано в `docs/CUSTOM_QUESTIONS.md` и `Другой-вопрос-или-шкала.md`). 2. **Непроцентная шкала (баллы 1–7):** подписи полос обновятся автоматически из листа «Кодирование», но для удаления знака `%` после среднего в карточке сверху потребуется убрать `+'%'` в `LiveWidget.html` (явно указано в документации).
valalav commented 2026-11-09 18:53:48 +00:00
Author
Owner
Copy link

🤖 gemini-4.0-argon · DESKTOP-6KC7IA7

Дополнение по п. 3 (#8, коммит 332c567): при проверке в Playwright (1366x768 и 390x844) на длинных русских словах в пользовательской шкале («Незначительно снизится», «непродовольственные товары») выявлено и устранено горизонтальное переполнение .label (scrollWidth > clientWidth). В src/LiveWidget.html добавлены overflow-wrap:anywhere;word-break:break-word, адаптивная ширина колонки подписи --label-col (260px при длине подписи > 14 символов на десктопе, 140px для стандартной шкалы) и minmax(105px,38%) 1fr на мобильном экране (overlapX=False, overflowX=False по всем 7 строкам на обоих разрешениях).

🤖 **gemini-4.0-argon** · DESKTOP-6KC7IA7 Дополнение по п. 3 (#8, коммит `332c567`): при проверке в Playwright (`1366x768` и `390x844`) на длинных русских словах в пользовательской шкале («Незначительно снизится», «непродовольственные товары») выявлено и устранено горизонтальное переполнение `.label` (`scrollWidth > clientWidth`). В `src/LiveWidget.html` добавлены `overflow-wrap:anywhere;word-break:break-word`, адаптивная ширина колонки подписи `--label-col` (`260px` при длине подписи `> 14` символов на десктопе, `140px` для стандартной шкалы) и `minmax(105px,38%) 1fr` на мобильном экране (`overlapX=False, overflowX=False` по всем 7 строкам на обоих разрешениях).
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
codex/shared-service-it
codex/luna-survey-validation
codex/terra-6.1-survey
experiment/qwen-flash
experiment/gemini-flash
No results found.
Labels
Clear labels   
agent:gemini-3.8-flash

   
agent:gemini-4.0-argon

   
agent:mimo-2.6-pro

   
state:doing

   
state:review
No labels
agent:gemini-3.8-flash
agent:gemini-4.0-argon
agent:mimo-2.6-pro
state:doing
state:review
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
valalav/ks-live-survey#8
No description provided.