Improve space validation and layout

.gitignore

@@ -0,0 +1,2 @@
+__pycache__/
+*.py[cod]

app.py

@@ -44,7 +44,6 @@ CSS = """
   --page-text: #0f172a;
   --page-muted: #526075;
   --page-line: rgba(15, 23, 42, 0.12);
-  --page-surface: rgba(255, 255, 255, 0.78);
   --page-surface-strong: #ffffff;
 }
 
@@ -81,11 +80,20 @@ textarea {
 
 .page-shell {
   margin-top: 26px;
-  padding: 36px 48px 42px;
   background: #ffffff;
   border: 1px solid rgba(15, 23, 42, 0.08);
   border-radius: 22px;
   box-shadow: 0 18px 48px rgba(15, 23, 42, 0.05);
+  overflow: hidden;
+  padding: 34px 0 40px;
+}
+
+.page-shell-content {
+  gap: 0 !important;
+}
+
+.shell-spacer {
+  min-width: 44px !important;
 }
 
 .hero {
@@ -201,15 +209,23 @@ textarea {
 }
 
 .main-form {
-  padding-right: 18px;
+  padding-right: 0;
 }
 
 .side-notes {
-  padding-left: 18px;
+  padding-left: 0;
+}
+
+.main-form > div {
+  padding-right: 18px !important;
+}
+
+.side-notes > div {
+  padding-left: 18px !important;
 }
 
 .caption {
-  margin-top: 4px;
+  margin-top: 8px;
   color: var(--page-muted);
   font-size: 0.93rem;
   line-height: 1.6;
@@ -224,11 +240,15 @@ textarea {
 }
 
 .results-shell {
-  margin-top: 26px;
+  margin-top: 0;
   padding-top: 22px;
   border-top: 1px solid var(--page-line);
 }
 
+.results-shell > div {
+  margin-top: 26px;
+}
+
 .action-row {
   margin-top: 10px;
 }
@@ -330,23 +350,17 @@ textarea {
   color: var(--page-text) !important;
 }
 
-.link-list a {
-  color: #1346a2;
-  text-decoration: none;
-  font-weight: 600;
-}
-
-.link-list a:hover {
-  text-decoration: underline;
-}
-
 @media (max-width: 900px) {
   .gradio-container {
     padding: 22px 18px 42px !important;
   }
 
   .page-shell {
-    padding: 26px 22px 30px;
+    padding: 24px 0 30px;
+  }
+
+  .shell-spacer {
+    min-width: 18px !important;
   }
 
   .hero {
@@ -410,6 +424,9 @@ your_submission.zip
     ├── task_info.json
     ├── data/
     ├── related_work/
+    │   ├── paper_000.pdf
+    │   ├── paper_001.pdf
+    │   └── ...
     └── target_study/
         ├── checklist.json
         ├── paper.pdf
@@ -495,6 +512,7 @@ def validate_submission(
         return None, '', '## Validation failed\n\n- Please upload a zip file.', '{}', gr.update(interactive=False), ''
 
     domain = resolve_domain(suggested_domain, custom_domain)
+    token = load_hf_token()
     metadata = SubmissionMetadata(
         domain=domain,
         submitter=submitter,
@@ -505,10 +523,10 @@ def validate_submission(
     )
 
     try:
-        existing_ids = list_existing_task_ids(repo_id=DEFAULT_REPO_ID, token=load_hf_token())
+        existing_ids = list_existing_task_ids(repo_id=DEFAULT_REPO_ID, token=token)
         assigned_task_id = allocate_next_task_id(domain, existing_ids)
         prepared = validate_and_prepare_submission(archive_path, metadata, assigned_task_id)
-        pr_ready = bool(load_hf_token())
+        pr_ready = bool(token)
         return (
             prepared.to_state(),
             prepared.assigned_task_id,
@@ -539,18 +557,22 @@ def validate_submission(
 
 def create_pr(state: dict | None):
     if not state:
-        return '## PR creation failed\n\n- Validate a submission first.'
+        return None, gr.update(interactive=False), '## PR creation failed\n\n- Validate a submission first.'
 
     prepared = PreparedSubmission.from_state(state)
+    token = load_hf_token()
     try:
-        commit_info = create_dataset_pr(prepared, repo_id=DEFAULT_REPO_ID, token=load_hf_token())
+        commit_info = create_dataset_pr(prepared, repo_id=DEFAULT_REPO_ID, token=token)
         pr_url = commit_info.pr_url or commit_info.commit_url
-        return '\n'.join([
+        message = '\n'.join([
             '## PR created',
             '',
             f'- Task ID: `{prepared.assigned_task_id}`',
             f'- PR: {pr_url}',
         ])
+        return None, gr.update(interactive=False), message
+    except Exception as exc:
+        return None, gr.update(interactive=False), build_failure_markdown(str(exc).strip() or 'Unknown PR creation error')
     finally:
         cleanup_work_dir(prepared.work_dir)
 
@@ -562,85 +584,91 @@ with gr.Blocks(title=SPACE_TITLE, fill_width=True) as demo:
     gr.HTML(build_hero_html())
 
     with gr.Group(elem_classes=['page-shell']):
-        with gr.Row(elem_classes=['section-row']):
-            with gr.Column(scale=7, elem_classes=['section-copy', 'main-form']):
-                gr.HTML(field_label_html('Task ZIP archive'))
-                with gr.Row(elem_classes=['upload-row']):
-                    archive = gr.UploadButton(
-                        'Select ZIP file',
-                        file_types=['.zip'],
-                        file_count='single',
-                        type='filepath',
-                        variant='secondary',
-                        elem_classes=['upload-button'],
-                    )
-                archive_notice = gr.Markdown('No ZIP file selected yet.', elem_classes=['upload-status'])
-                with gr.Row():
-                    with gr.Column():
-                        gr.HTML(field_label_html('Suggested domain'))
-                        suggested_domain = gr.Dropdown(
-                            choices=list(DOMAINS),
-                            value='Astronomy',
+        with gr.Row():
+            with gr.Column(scale=1, min_width=0, elem_classes=['shell-spacer']):
+                gr.HTML('')
+            with gr.Column(scale=30, min_width=0, elem_classes=['page-shell-content']):
+                with gr.Row(elem_classes=['section-row']):
+                    with gr.Column(scale=7, elem_classes=['section-copy', 'main-form']):
+                        gr.HTML(field_label_html('Task ZIP archive'))
+                        with gr.Row(elem_classes=['upload-row']):
+                            archive = gr.UploadButton(
+                                'Select ZIP file',
+                                file_types=['.zip'],
+                                file_count='single',
+                                type='filepath',
+                                variant='secondary',
+                                elem_classes=['upload-button'],
+                            )
+                        archive_notice = gr.Markdown('No ZIP file selected yet.', elem_classes=['upload-status'])
+                        with gr.Row():
+                            with gr.Column():
+                                gr.HTML(field_label_html('Suggested domain'))
+                                suggested_domain = gr.Dropdown(
+                                    choices=list(DOMAINS),
+                                    value='Astronomy',
+                                    show_label=False,
+                                    container=False,
+                                )
+                            with gr.Column():
+                                gr.HTML(field_label_html('Custom domain (optional)'))
+                                custom_domain = gr.Textbox(
+                                    placeholder='e.g. Robotics or Robot-Learning',
+                                    show_label=False,
+                                    container=False,
+                                )
+                        gr.Markdown(
+                            '<div class="caption">Use the custom field if your task does not belong to the suggested list. '
+                            'If the custom field is filled, it overrides the suggested domain and becomes the prefix of the final task ID.</div>'
+                        )
+                        gr.HTML(field_label_html('Submitter name or HF username'))
+                        submitter = gr.Textbox(
+                            placeholder='e.g. your-hf-handle',
                             show_label=False,
                             container=False,
                         )
-                    with gr.Column():
-                        gr.HTML(field_label_html('Custom domain (optional)'))
-                        custom_domain = gr.Textbox(
-                            placeholder='e.g. Robotics or Robot-Learning',
+                        gr.HTML(field_label_html('Contact email'))
+                        email = gr.Textbox(
+                            placeholder='name@example.com',
                             show_label=False,
                             container=False,
                         )
-                gr.Markdown(
-                    '<div class="caption">Use the custom field if your task does not belong to the suggested list. '
-                    'If the custom field is filled, it overrides the suggested domain and becomes the prefix of the final task ID.</div>'
-                )
-                gr.HTML(field_label_html('Submitter name or HF username'))
-                submitter = gr.Textbox(
-                    placeholder='e.g. your-hf-handle',
-                    show_label=False,
-                    container=False,
-                )
-                gr.HTML(field_label_html('Contact email'))
-                email = gr.Textbox(
-                    placeholder='name@example.com',
-                    show_label=False,
-                    container=False,
-                )
-                gr.HTML(field_label_html('Target paper title'))
-                paper_title = gr.Textbox(show_label=False, container=False)
-                gr.HTML(field_label_html('Target paper URL or DOI'))
-                paper_url = gr.Textbox(
-                    placeholder='https://... or DOI',
-                    show_label=False,
-                    container=False,
-                )
-                gr.HTML(field_label_html('Optional notes for reviewers'))
-                notes = gr.Textbox(
-                    lines=4,
-                    placeholder='Anything maintainers should know about licensing, preprocessing, or provenance.',
-                    show_label=False,
-                    container=False,
-                )
-            with gr.Column(scale=5, elem_classes=['section-copy', 'side-notes']):
-                gr.Markdown(submission_guide_markdown(), elem_classes=['subtle-block'])
-
-        with gr.Row(elem_classes=['action-row']):
-            validate_btn = gr.Button('Validate ZIP', variant='primary', elem_classes=['primary-button'])
-            create_pr_btn = gr.Button('Create Dataset PR', interactive=False, elem_classes=['secondary-button'])
-
-        with gr.Column(elem_classes=['section-copy', 'results-shell']):
-            gr.HTML(field_label_html('Final task ID (assigned automatically)'))
-            assigned_task_id = gr.Textbox(
-                interactive=False,
-                show_label=False,
-                container=False,
-            )
-            gr.Markdown(final_task_help_html())
-            validation_md = gr.Markdown()
-            gr.HTML(field_label_html('Validation report'))
-            validation_report = gr.Code(language='json', show_label=False, container=False)
-            pr_md = gr.Markdown()
+                        gr.HTML(field_label_html('Target paper title'))
+                        paper_title = gr.Textbox(show_label=False, container=False)
+                        gr.HTML(field_label_html('Target paper URL or DOI'))
+                        paper_url = gr.Textbox(
+                            placeholder='https://... or DOI',
+                            show_label=False,
+                            container=False,
+                        )
+                        gr.HTML(field_label_html('Optional notes for reviewers'))
+                        notes = gr.Textbox(
+                            lines=4,
+                            placeholder='Anything maintainers should know about licensing, preprocessing, or provenance.',
+                            show_label=False,
+                            container=False,
+                        )
+                    with gr.Column(scale=5, elem_classes=['section-copy', 'side-notes']):
+                        gr.Markdown(submission_guide_markdown(), elem_classes=['subtle-block'])
+
+                with gr.Row(elem_classes=['action-row']):
+                    validate_btn = gr.Button('Validate ZIP', variant='primary', elem_classes=['primary-button'])
+                    create_pr_btn = gr.Button('Create Dataset PR', interactive=False, elem_classes=['secondary-button'])
+
+                with gr.Column(elem_classes=['section-copy', 'results-shell']):
+                    gr.HTML(field_label_html('Final task ID (assigned automatically)'))
+                    assigned_task_id = gr.Textbox(
+                        interactive=False,
+                        show_label=False,
+                        container=False,
+                    )
+                    gr.Markdown(final_task_help_html())
+                    validation_md = gr.Markdown()
+                    gr.HTML(field_label_html('Validation report'))
+                    validation_report = gr.Code(language='json', show_label=False, container=False)
+                    pr_md = gr.Markdown()
+            with gr.Column(scale=1, min_width=0, elem_classes=['shell-spacer']):
+                gr.HTML('')
 
     archive.upload(fn=handle_archive_upload, inputs=[archive], outputs=[archive_state, archive_notice])
 
@@ -659,7 +687,7 @@ with gr.Blocks(title=SPACE_TITLE, fill_width=True) as demo:
         ],
         outputs=[state, assigned_task_id, validation_md, validation_report, create_pr_btn, pr_md],
     )
-    create_pr_btn.click(fn=create_pr, inputs=[state], outputs=[pr_md])
+    create_pr_btn.click(fn=create_pr, inputs=[state], outputs=[state, create_pr_btn, pr_md])
 
 
 if __name__ == '__main__':

requirements.txt

@@ -1 +1 @@
-huggingface_hub>=0.34.0,<1
+huggingface_hub>=1.3.0,<2

validator.py

@@ -317,6 +317,10 @@ def validate_task_dir(
         errors.append(str(exc))
         return errors
 
+    if not isinstance(task_info, dict):
+        errors.append('task_info.json root must be a JSON object')
+        return errors
+
     if tuple(sorted(task_info.keys())) != EXPECTED_TASK_INFO_KEYS:
         errors.append(f'task_info.json keys mismatch: {sorted(task_info.keys())}')
 
@@ -326,10 +330,12 @@ def validate_task_dir(
         errors.append('task_info.json field `data` must be a list')
         task_info['data'] = []
 
-    covered_files: set[Path] = set()
     declared_paths: set[str] = set()
     for idx, item in enumerate(task_info['data']):
         prefix = f'task_info.data[{idx}]'
+        if not isinstance(item, dict):
+            errors.append(f'{prefix} must be an object')
+            continue
         if tuple(sorted(item.keys())) != EXPECTED_DATA_ITEM_KEYS:
             errors.append(f'{prefix} keys mismatch: {sorted(item.keys())}')
             continue
@@ -354,27 +360,16 @@ def validate_task_dir(
         if not target.exists():
             errors.append(f'{prefix}.path does not exist: {data_path}')
             continue
-        if target.is_file():
-            covered_files.add(target)
-        elif target.is_dir():
+        if target.is_dir():
             nested_files = {p for p in target.rglob('*') if p.is_file()}
             if not nested_files:
                 errors.append(f'{prefix}.path points to an empty directory: {data_path}')
-            covered_files.update(nested_files)
-        else:
+        elif not target.is_file():
             errors.append(f'{prefix}.path is neither file nor directory: {data_path}')
         description = item.get('description', '')
         if any(token in description for token in STALE_TOKENS):
             errors.append(f'{prefix}.description still contains stale source paths or legacy directories')
 
-    actual_data_files = _iter_visible_files(data_dir) if data_dir.exists() else set()
-    uncovered = sorted(actual_data_files - covered_files)
-    if uncovered:
-        errors.append('data/ contains undeclared files: ' + ', '.join(rel(p, task_dir) for p in uncovered[:20]))
-    missing_backing = sorted(covered_files - actual_data_files)
-    if missing_backing:
-        errors.append('declared data coverage points outside data/: ' + ', '.join(rel(p, task_dir) for p in missing_backing[:20]))
-
     related_entries = sorted(related_dir.iterdir(), key=lambda p: p.name) if related_dir.exists() else []
     related_files = [p for p in related_entries if p.is_file()]
     related_dirs = [p for p in related_entries if p.is_dir()]
@@ -400,13 +395,20 @@ def validate_task_dir(
         errors.append(str(exc))
         return errors
 
-    if not isinstance(checklist, list) or not checklist:
+    if not isinstance(checklist, list):
+        errors.append('checklist.json root must be a list')
+        return errors
+
+    if not checklist:
         errors.append('checklist.json must be a non-empty list')
         checklist = []
 
     referenced_images: set[str] = set()
     for idx, item in enumerate(checklist):
         prefix = f'checklist[{idx}]'
+        if not isinstance(item, dict):
+            errors.append(f'{prefix} must be an object')
+            continue
         if tuple(sorted(item.keys())) != EXPECTED_CHECKLIST_ITEM_KEYS:
             errors.append(f'{prefix} keys mismatch: {sorted(item.keys())}')
             continue
@@ -437,10 +439,7 @@ def validate_task_dir(
                 referenced_images.add(path_value)
 
     actual_image_files = {str(p.relative_to(target_dir)) for p in _iter_visible_files(images_dir)} if images_dir.exists() else set()
-    extra_images = sorted(actual_image_files - referenced_images)
     missing_images = sorted(referenced_images - actual_image_files)
-    if extra_images:
-        errors.append('target_study/images contains unreferenced files: ' + ', '.join(extra_images[:20]))
     if missing_images:
         errors.append('checklist image references are missing from target_study/images: ' + ', '.join(missing_images[:20]))